[PATCH] D86050: [SCEV] Refactor isHighCostExpansionHelper

[Sam Parker via Phabricator via llvm-commits]

samparker updated this revision to Diff 286028.
samparker added a comment.
Herald added a subscriber: zzheng.

- Extracted the SCEV changes and rebased.
- Removed the use of unique_pointer.
- Moved the cost adjustment for NAry operands into the helper. These now also use the extra operand checks that were performed for AddRecs.


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

https://reviews.llvm.org/D86050

Files:
  llvm/include/llvm/Transforms/Utils/ScalarEvolutionExpander.h
  llvm/lib/Transforms/Utils/ScalarEvolutionExpander.cpp
  llvm/test/Transforms/IndVarSimplify/X86/loop-invariant-conditions.ll
  llvm/test/Transforms/IndVarSimplify/widen-loop-comp.ll
  llvm/test/Transforms/LoopUnroll/runtime-small-upperbound.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D86050.286028.patch
Type: text/x-patch
Size: 27024 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200817/d83f1491/attachment.bin>