[PATCH] D86346: [SimplifyCFG] Accumulate cost against budget

[Roman Lebedev via Phabricator via llvm-commits]

lebedev.ri added inline comments.


================
Comment at: llvm/lib/Transforms/Utils/SimplifyCFG.cpp:2138
       ++SpeculatedInstructions;
       if (SpeculatedInstructions > 1)
         return false;
----------------
Ah, so this transform basically never fires then,


================
Comment at: llvm/lib/Transforms/Utils/SimplifyCFG.cpp:2170
+
+    BudgetRemaining -= OrigCost - ThenCost;
+    if (BudgetRemaining < 0)
----------------
I'm not sure what is going on here.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D86346