[PATCH] D110817: [GVN] Simple GVN hoist - scalars

[Momchil Velikov via Phabricator via llvm-commits]

chill marked 2 inline comments as done.
chill added inline comments.


================
Comment at: llvm/lib/Transforms/Scalar/GVN.cpp:3062
+static bool isHoistBarrier(const Instruction &I) {
+  return !isGuaranteedToTransferExecutionToSuccessor(&I);
+}
----------------
mkazantsev wrote:
> Re-reading the patch, I noticed that it might be missing aliasing notion. How do we prevent hoisting loads across aliasing store?
In this part we don't hoist loads and stores at all, since they have unique value numbers.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D110817/new/

https://reviews.llvm.org/D110817