[PATCH] D89818: [Compile Time] Make it possible to skip redundant debuginst removal

Adrian Prantl via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 20 13:26:13 PDT 2020


aprantl added a comment.

Is this an NFC patch? If not, can you please add a test case?



================
Comment at: llvm/include/llvm/Transforms/Utils/BasicBlockUtils.h:93
 /// successor Sing. In this case the branch will be updated with Sing instead of
 /// BB, and BB will still be merged into its predecessor and removed.
 bool MergeBlockIntoPredecessor(BasicBlock *BB, DomTreeUpdater *DTU = nullptr,
----------------
please document under what circumstances one would pass true to the last two parameters


================
Comment at: llvm/lib/Transforms/Utils/LoopUnroll.cpp:867
+      if (MergeBlockIntoPredecessor(Dest, &DTU, LI, nullptr, nullptr, false,
+                                    true)) {
         // Dest has been folded into Fold. Update our worklists accordingly.
----------------
please add a comment that explains why passing true is correct/needed/right here.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D89818



More information about the llvm-commits mailing list