[PATCH] Further relax the constraint on atomics in MemoryDependencyAnalysis.cpp
JF Bastien
jfb at chromium.org
Wed Aug 20 21:53:21 PDT 2014
The change looks good overall, but I'd like to have @reames review it too.
================
Comment at: test/Transforms/DeadStoreElimination/atomic.ll:163
@@ +162,3 @@
+ store atomic i32 0, i32* @y release, align 4
+ %x = load atomic i32* @y acquire, align 4
+ store i32 1, i32* @x
----------------
Why not also test load-acq followed by store-rel here and in other places (or the reverse store/load)? It seems like a good sanity check.
================
Comment at: test/Transforms/GVN/atomic.ll:133
@@ -106,1 +132,2 @@
+
----------------
Drop the extra space.
http://reviews.llvm.org/D4845
More information about the llvm-commits
mailing list