[PATCH] D92238: [SCEVExpander] Migrate costAndCollectOperands to use InstructionCost.
Roman Lebedev via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Jan 18 08:03:00 PST 2021
lebedev.ri added inline comments.
================
Comment at: llvm/lib/Transforms/Utils/ScalarEvolutionExpander.cpp:2348-2349
SmallVectorImpl<SCEVOperand> &Worklist) {
- if (BudgetRemaining < 0)
+ if (!BudgetRemaining.isValid() || BudgetRemaining < 0)
return true; // Already run out of budget, give up.
----------------
What happens without the `isValid()` check?
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D92238/new/
https://reviews.llvm.org/D92238
More information about the llvm-commits
mailing list