[PATCH] D91174: [Analysis] Introduce a new InstructionCost class

[David Sherwood via Phabricator via llvm-commits]

david-arm updated this revision to Diff 306107.
david-arm added a comment.

- Added comments explaining the reason for the behaviour used in the comparison operators.
- Removed the comment above operator-.
- Added private propagateState() function, which is more generic and will support more than state. It uses the same lexicographical ordering as the comparisons.
- Updated the tests to use more readable variable names.


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

https://reviews.llvm.org/D91174

Files:
  llvm/include/llvm/Analysis/InstructionCost.h
  llvm/include/llvm/Analysis/TargetTransformInfo.h
  llvm/include/llvm/IR/DiagnosticInfo.h
  llvm/lib/Analysis/CostModel.cpp
  llvm/lib/Analysis/TargetTransformInfo.cpp
  llvm/lib/CodeGen/InterleavedLoadCombinePass.cpp
  llvm/lib/IR/DiagnosticInfo.cpp
  llvm/lib/Transforms/IPO/HotColdSplitting.cpp
  llvm/lib/Transforms/Scalar/CallSiteSplitting.cpp
  llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp
  llvm/unittests/Analysis/CMakeLists.txt
  llvm/unittests/Analysis/InstructionCostTest.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D91174.306107.patch
Type: text/x-patch
Size: 18202 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20201118/c8723d97/attachment.bin>