[PATCH] D12992: invariant.group handling in GVN

[Piotr Padlewski via llvm-commits]

Prazek added inline comments.

================
Comment at: lib/Analysis/MemoryDependenceAnalysis.cpp:419
@@ +418,3 @@
+        Seen.insert(BCI->getOperand(0));
+      }
+    }
----------------
it doesn't matter if we will return Def for A or B, because they will load the same value. The second thing is that we will process them in order which they are in the block, which means that for A we will do nothing, for B we will return Def(A) and remove B, and for C the same thing (because B will not longer exist)


http://reviews.llvm.org/D12992