[PATCH] D44160: [GVN] Don't use the eliminated load as an available value in phi construction

[Philip Reames via Phabricator via llvm-commits]

reames added a comment.

Sorry, thought I'd reviewed this.  I apparently didn't hit submit.



================
Comment at: lib/Transforms/Scalar/GVN.cpp:769
 
+    // If the value is the load that we will be eliminating, and the block it's
+    // available in is the block that the load is in, then don't add it as
----------------
Don't you need to add the original LI to the SSA updater at least once?  Starting at the original block?


Repository:
  rL LLVM

https://reviews.llvm.org/D44160