[llvm] [FuncSpec] Query SCCPSolver in more places (PR #114964)

Alexandros Lamprineas via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 5 03:01:45 PST 2024


================
@@ -91,12 +91,14 @@ static cl::opt<bool> SpecializeLiteralConstant(
         "argument"));
 
 bool InstCostVisitor::canEliminateSuccessor(BasicBlock *BB, BasicBlock *Succ,
-                                         DenseSet<BasicBlock *> &DeadBlocks) {
+                                            DenseSet<BasicBlock *> &DeadBlocks,
+                                            const SCCPSolver &Solver) {
----------------
labrinea wrote:

Is this not a member variable? Similarly DeadBlocks. It doesn't seem like the function needs to be static.

https://github.com/llvm/llvm-project/pull/114964


More information about the llvm-commits mailing list