<div dir="ltr">I will add the test with ppc440.<div><br></div><div>This path is only for fences, and I do not think a monotonic fence makes much sense. So it is not special cased (but should trigger a bunch of errors earlier in the front-end or at least asserts in the middle-end).</div><div><br></div><div>Thanks for the reviews !</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Oct 2, 2014 at 9:14 PM, <a href="mailto:hfinkel@anl.gov">hfinkel@anl.gov</a> <span dir="ltr"><<a href="mailto:hfinkel@anl.gov" target="_blank">hfinkel@anl.gov</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">LGTM.<br>
<br>
Please add a test run with ppc440 to make sure we still get msync.<br>
<br>
================<br>
Comment at: lib/Target/PowerPC/PPCInstrInfo.td:2559<br>
@@ -2553,1 +2558,3 @@<br>
+def : Pat<(atomic_fence (i32 7), (imm)), (SYNC 0)>, Requires<[HasSYNC]>;<br>
+def : Pat<(atomic_fence (imm),   (imm)), (SYNC 1)>, Requires<[HasSYNC]>;<br>
 def : Pat<(atomic_fence (imm), (imm)), (MSYNC)>, Requires<[HasOnlyMSYNC]>;<br>
----------------<br>
By the way, what happens to AtomicOrdering == 2 (which is Monotonic). Is that (should that be) a noop?<br>
<br>
<a href="http://reviews.llvm.org/D5317" target="_blank">http://reviews.llvm.org/D5317</a><br>
<br>
<br>
</blockquote></div><br></div>