mkazantsev added a comment.
So now every single test needs to be regenerated? It'll create straw diff from
nowhere...
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D139006/new/
https://reviews.llvm.org/D139006
_
mkazantsev added inline comments.
Comment at: llvm/lib/Transforms/Scalar/SimplifyCFGPass.cpp:263
const SimplifyCFGOptions &Options) {
- assert((!RequireAndPreserveDomTree ||
- (DT && DT->verify(DominatorTree::VerificationLevel::Full))) &
mkazantsev added a comment.
This is awesome. And scary. This is scarily awesome. :)
Comment at: llvm/lib/Transforms/Scalar/SimplifyCFGPass.cpp:263
const SimplifyCFGOptions &Options) {
- assert((!RequireAndPreserveDomTree ||
- (DT && DT
This revision was not accepted when it landed; it landed in state "Needs
Review".
This revision was automatically updated to reflect the committed changes.
Closed by commit rG360ab707127d: [SimplifyCFG] Do not create unneeded PR Phi in
block with convergent calls (authored by mkazantsev).
Herald
This revision was automatically updated to reflect the committed changes.
Closed by commit rG360ab707127d: [SimplifyCFG] Do not create unneeded PR Phi in
block with convergent calls (authored by mkazantsev).
Herald added a project: clang.
Herald added a subscriber: cfe-commits.
Repository:
rG L