Add generic costs calculation for SADDSAT/SSUBSAT intrinsics, this uses generic costs for sadd_with_overflow/ssub_with_overflow, an extra sign comparison + a selects based on the sign/overflow.
This completes PR40316
Paths
| Differential D57239
[TTI] Add generic SADDSAT/SSUBSAT costs ClosedPublic Authored by RKSimon on Jan 25 2019, 6:51 AM.
Details Summary Add generic costs calculation for SADDSAT/SSUBSAT intrinsics, this uses generic costs for sadd_with_overflow/ssub_with_overflow, an extra sign comparison + a selects based on the sign/overflow. This completes PR40316
Diff Detail
Event TimelineComment Actions LGTM, assuming that we're going to drop the scalarization for vector sadd.add/sub in the near feature. This revision is now accepted and ready to land.Jan 26 2019, 12:25 PM Closed by commit rL352315: [TTI] Add generic SADDSAT/SSUBSAT costs (authored by RKSimon). · Explain WhyJan 27 2019, 5:52 AM This revision was automatically updated to reflect the committed changes.
Revision Contents
Diff 183532 include/llvm/CodeGen/BasicTTIImpl.h
test/Analysis/CostModel/X86/arith-ssat.ll
test/Transforms/SLPVectorizer/X86/arith-add-ssat.ll
test/Transforms/SLPVectorizer/X86/arith-sub-ssat.ll
|