[PATCH] D86050: [SCEV] Refactor isHighCostExpansionHelper

Sam Parker via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 17 01:41:46 PDT 2020


samparker created this revision.
samparker added reviewers: lebedev.ri, reames, mkazantsev.
Herald added subscribers: javed.absar, hiraditya.
Herald added a project: LLVM.
samparker requested review of this revision.

As discussed in D76434 <https://reviews.llvm.org/D76434>, to enable the cost of constants, the helper function has been reorganised:

- A struct has been introduced to hold SCEV operand information so that we know the user of the operand, as well as the operand index. The Worklist now uses instead instead of a bare SCEV.
- The costing of each SCEV, and collection of its operands, is now performed in a helper function.
- Some SCEVExprs have been modified to provide methods that allow us to iterate through their operands.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D86050

Files:
  llvm/include/llvm/Analysis/ScalarEvolutionExpressions.h
  llvm/include/llvm/Transforms/Utils/ScalarEvolutionExpander.h
  llvm/lib/Transforms/Utils/ScalarEvolutionExpander.cpp

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


More information about the llvm-commits mailing list