[PATCH] Mutate TargetLowering::shouldExpandAtomicRMWInIR to specifically dictate how AtomicRMWInsts are expanded.

JF Bastien jfb at chromium.org
Wed Feb 18 10:49:14 PST 2015


Looks good overall, but I'll let @t.p.northover comment more.


REPOSITORY
  rL LLVM

================
Comment at: lib/CodeGen/AtomicExpandPass.cpp:218
@@ -217,3 +217,1 @@
-  // It is the responsibility of the target to only return true in
-  // shouldExpandAtomicRMW in cases where this is required and possible.
   IRBuilder<> Builder(SI);
----------------
Why remove this?

http://reviews.llvm.org/D7713

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list