<div dir="ltr"><div><div><div>Hi Jeehoon<br><br></div>I looked at the original code review <a href="http://reviews.llvm.org/D5180">http://reviews.llvm.org/D5180</a> and checked with Bill Schmidt who approved the code change. The only reason that the improvement was not done in the original commit was lack of time. Please go ahead with your change. Make sure to add Hal Finkel (PowerPC Backend maintainer) in your code review.<br><br></div>Thanks<br></div>Ehsan<br><br></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Jun 2, 2016 at 11:16 PM, Jeehoon Kang via llvm-dev <span dir="ltr"><<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi everyone,<div><br></div><div>I wonder if why acquire loads (atomic accesses) are compiled to PowerPC as `load+lwsync`, which is stronger than what is known to be sound: `load+ctrl+isync`.  The relevant code section is: <a href="https://github.com/llvm-mirror/llvm/blob/master/lib/Target/PowerPC/PPCISelLowering.cpp#L8374" target="_blank">https://github.com/llvm-mirror/llvm/blob/master/lib/Target/PowerPC/PPCISelLowering.cpp#L8374</a></div><div><br></div><div>Note that the compiler writers recognized this issue, and there is a comment on it in the above code section.  But I would like to know if there are more discussions on this issue.  Specifically, I would like to know if the current compilation scheme is intentionally used.  Otherwise, I want to make a patch that weakens the compilation scheme of acquire loads.</div><div><br></div><div>Thank you,</div><div>Jeehoon</div><span class="HOEnZb"><font color="#888888"><div><div><br></div>-- <br><div data-smartmail="gmail_signature"><div dir="ltr"><a href="http://sf.snu.ac.kr/jeehoon.kang" target="_blank">Jeehoon Kang (Ph.D. student)</a><div><a href="http://sf.snu.ac.kr" target="_blank">Software Foundations Laboratory</a><div><a href="http://www.snu.ac.kr" target="_blank">Seoul National University</a></div></div></div></div>
</div></font></span></div>
<br>_______________________________________________<br>
LLVM Developers mailing list<br>
<a href="mailto:llvm-dev@lists.llvm.org">llvm-dev@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev</a><br>
<br></blockquote></div><br></div>