[PATCH] D66604: [GVN] AnalyzeLoadAvailability: Replace a load after lifetime.end with undef (PR20811)
Florian Hahn via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Sep 12 13:32:45 PDT 2019
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
More information about the llvm-commits
mailing list