[PATCH] D76434: [SCEV] Query expanded immediate cost at minsize

[Sjoerd Meijer via Phabricator via llvm-commits]

SjoerdMeijer added a comment.

Nice one!



================
Comment at: llvm/lib/Transforms/Utils/ScalarEvolutionExpander.cpp:2408
     // And finally, the operands themselves should fit within the budget.
-    Worklist.insert(Worklist.end(), NAry->operands().begin(),
-                    NAry->operands().end());
+    if (CostKind == TTI::TCK_CodeSize) {
+      unsigned Idx = 0;
----------------
This change here looks like an exact duplication of the change above (lines 2355 - 2362). Can this be in a helper?


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

https://reviews.llvm.org/D76434