[PATCH] D66604: [GVN] AnalyzeLoadAvailability: Replace a load after lifetime.end with undef (PR20811)

[Florian Hahn via Phabricator via llvm-commits]

fhahn added a comment.

Thanks! Could you also add another test where we have a call to lifetime.end, followed by another defining store, followed by a load? This should not get optimized by this patch.



================
Comment at: llvm/lib/Transforms/Scalar/GVN.cpp:873
   Instruction *DepInst = DepInfo.getInst();
+
   if (DepInfo.isClobber()) {
----------------
please remove the unrelated whitespace change.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D66604