[PATCH] D110739: [CostModel] Update default cost model for sadd/ssub overflow to match TargetLowering

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 29 15:10:08 PDT 2021


craig.topper updated this revision to Diff 376051.
craig.topper added a comment.

Use correct ICMP predicate


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

https://reviews.llvm.org/D110739

Files:
  llvm/include/llvm/CodeGen/BasicTTIImpl.h
  llvm/test/Analysis/CostModel/AArch64/arith-ssat.ll
  llvm/test/Analysis/CostModel/ARM/arith-overflow.ll
  llvm/test/Analysis/CostModel/ARM/arith-ssat.ll
  llvm/test/Analysis/CostModel/X86/arith-overflow.ll
  llvm/test/Analysis/CostModel/X86/arith-ssat.ll
  llvm/test/Transforms/LoopVectorize/AArch64/intrinsiccost.ll
  llvm/test/Transforms/LoopVectorize/ARM/mve-saddsatcost.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D110739.376051.patch
Type: text/x-patch
Size: 314418 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210929/acc59111/attachment-0001.bin>


More information about the llvm-commits mailing list