[PATCH] D22326: [JumpThreading] PRE unordered loads

[Sanjoy Das via llvm-commits]

sanjoy added a comment.

Will add the negative tests as you suggest.


================
Comment at: lib/Transforms/Scalar/JumpThreading.cpp:1059
@@ +1058,3 @@
+    LoadInst *NewVal =
+        new LoadInst(LoadedPtr, LI->getName() + ".pr", /*isVolatile*/ false,
+                     LI->getAlignment(), LI->getOrdering(), LI->getSynchScope(),
----------------
jfb wrote:
> Li->isVolatile() seems simpler than having the comment.
That sounds like an unrelated (to this patch) change.  I've removed the `/*isVolatile*/` comment for now (so that bit stays the same), let me know if you'd like me to `s/false/LI->isVolatile()/` separately.


https://reviews.llvm.org/D22326