[PATCH] D95351: NFC: Migrate SimplifyCFG to work on InstructionCost

[Sanjay Patel via Phabricator via llvm-commits]

spatel added inline comments.


================
Comment at: llvm/lib/Transforms/Utils/SimplifyCFG.cpp:338
 /// expensive.
-static unsigned ComputeSpeculationCost(const User *I,
-                                       const TargetTransformInfo &TTI) {
+static InstructionCost ComputeSpeculationCost(const User *I,
+                                              const TargetTransformInfo &TTI) {
----------------
Here and below: since we're updating function signatures, it would make sense to update the formatting as clang-tidy suggests:
computeSpeculationCost()

This can be done as an NFC preliminary commit.


================
Comment at: llvm/lib/Transforms/Utils/SimplifyCFG.cpp:364
                                 SmallPtrSetImpl<Instruction *> &AggressiveInsts,
-                                int &BudgetRemaining,
+                                InstructionBudget &BudgetRemaining,
                                 const TargetTransformInfo &TTI,
----------------
Which patch defines `InstructionBudget`?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D95351