[PATCH] D24844: GVN-hoist: disable hoisting stores past calls (PR30499)

[Hal Finkel via llvm-commits]

hfinkel added a subscriber: hfinkel.

================
Comment at: llvm/lib/Transforms/Scalar/GVNHoist.cpp:368
@@ -363,1 +367,3 @@
+
+      if (!AA->isNoAlias(DefLoc, MemoryLocation::get(Insn)))
         return true;
----------------
You could do something like:

  if (AA->getModRef(Insn, DefLoc) != MRI_NoModRef)
    return true;

and that should handle calls.


https://reviews.llvm.org/D24844