<html><body><p><font size="2">Thanks for your reply.</font><br><br><font size="2">Patch is created </font><a href="https://reviews.llvm.org/D55499"><font size="2">https://reviews.llvm.org/D55499</font></a><font size="2">.</font><br><br><font size="2">Thanks.</font><br><br><font size="2">BRS//</font><br><font size="2">Chen Zheng</font><br><font size="2">Power Compiler Backend Developer</font><br><br><br><img width="16" height="16" src="cid:1__=C7BB09CCDF9A88358f9e8a93df938690918cC7B@" border="0" alt="Inactive hide details for Tim Northover ---2018/12/08 06:07:28 PM---On Fri, 7 Dec 2018 at 18:25, Zheng CZ Chen via llvm-dev <ll"><font size="2" color="#424282">Tim Northover ---2018/12/08 06:07:28 PM---On Fri, 7 Dec 2018 at 18:25, Zheng CZ Chen via llvm-dev <llvm-dev@lists.llvm.org> wrote:</font><br><br><font size="2" color="#5F5F5F">From: </font><font size="2">Tim Northover <t.p.northover@gmail.com></font><br><font size="2" color="#5F5F5F">To: </font><font size="2">czhengsz@cn.ibm.com</font><br><font size="2" color="#5F5F5F">Cc: </font><font size="2">LLVM Developers Mailing List <llvm-dev@lists.llvm.org></font><br><font size="2" color="#5F5F5F">Date: </font><font size="2">2018/12/08 06:07 PM</font><br><font size="2" color="#5F5F5F">Subject: </font><font size="2">Re: [llvm-dev] Should intrinsics llvm.eh.sjlj.setjmp be with isBarrier flag?</font><br><hr width="100%" size="2" align="left" noshade style="color:#8091A5; "><br><br><br><tt><font size="2">On Fri, 7 Dec 2018 at 18:25, Zheng CZ Chen via llvm-dev<br><llvm-dev@lists.llvm.org> wrote:<br>> Is there any misunderstanding? Is there any specification reason for setting llvm.eh.sjlj.setjmp as a barrier?<br><br>I agree with you, the setjmp ones shouldn't be isBarrier. I suspect<br>the original commit mistook the name "isBarrier" as referring to a<br>memory fence-like construct. It's a fairly common mistake and could<br>easily apply here.<br><br>Cheers.<br><br>Tim.<br><br></font></tt><br><br><BR>
</body></html>