[PATCH] D93049: [SLPVectorizer]Migrate getTreeCost() to use InstructionCost

Sander de Smalen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 10 09:20:46 PST 2020


sdesmalen added inline comments.


================
Comment at: llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp:6201
   bool CandidateFound = false;
-  int MinCost = SLPCostThreshold;
+  InstructionCost MinCost = -SLPCostThreshold;
 
----------------
Where does the `-` come from in front of SLPCostThreshold?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D93049



More information about the llvm-commits mailing list