[PATCH] D70593: [LCSSA] Don't use VH callbacks to invalidate SCEV when creating LCSSA phis
Daniil Suchkov via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Nov 22 03:57:45 PST 2019
DaniilSuchkov marked an inline comment as done.
DaniilSuchkov added inline comments.
================
Comment at: llvm/lib/Transforms/Scalar/SimpleLoopUnswitch.cpp:322
// nodes for values used in the no-longer-nested loop.
- formLCSSA(*OldContainingL, DT, &LI, nullptr);
+ formLCSSA(*OldContainingL, DT, &LI, SE);
----------------
This change is required to preserve current behavior in cases when it's valid: sometimes invalidation of SCEV through VH actually worked as intended.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D70593/new/
https://reviews.llvm.org/D70593
More information about the llvm-commits
mailing list