[PATCH] D132408: [SimplifyCFG] accumulate bonus insts cost

[Artem Belevich via Phabricator via llvm-commits]

tra added a comment.

Nice. I like the consolidated `CostTracker`.



================
Comment at: llvm/lib/Transforms/Utils/SimplifyCFG.cpp:7197
+                       ArrayRef<WeakVH> LoopHeaders,
+                       SimplifyCFGCostTracker *NumBonusInsts) {
   return SimplifyCFGOpt(TTI, DTU, BB->getModule()->getDataLayout(), LoopHeaders,
----------------
This needs to be renamed, too.


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

https://reviews.llvm.org/D132408