[PATCH] D149428: [BBUtils][NFC] Delete SplitBlockAndInsertIfThen with DT.

Joshua Cao via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 23 21:02:24 PDT 2023


caojoshua marked an inline comment as done.
caojoshua added inline comments.


================
Comment at: llvm/lib/Transforms/Utils/LibCallsShrinkWrap.cpp:504
-// Verify the dominator after we've updated it locally.
-  assert(!DT || DT->verify(DominatorTree::VerificationLevel::Fast));
-  return Changed;
----------------
foad wrote:
> What's the reason for removing this extra verification?
No good reason. Added it back.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D149428



More information about the llvm-commits mailing list