diff options
| author | Craig Topper <craig.topper@sifive.com> | 2021-09-29 10:21:44 -0700 |
|---|---|---|
| committer | Craig Topper <craig.topper@sifive.com> | 2021-09-30 09:41:14 -0700 |
| commit | 765348298c52bcff7c33d766cda954c3371a700f (patch) | |
| tree | bcd23333797aa1060ed289b25b921d5552b27c36 /llvm/lib/Analysis/BasicAliasAnalysis.cpp | |
| parent | 267bb194f3cedd88db0e7a5ea86169d669948e74 (diff) | |
| download | llvm-765348298c52bcff7c33d766cda954c3371a700f.zip llvm-765348298c52bcff7c33d766cda954c3371a700f.tar.gz llvm-765348298c52bcff7c33d766cda954c3371a700f.tar.bz2 | |
[CostModel] Update default cost model for sadd/ssub overflow to match TargetLowering
The expansion for these was updated in https://reviews.llvm.org/D47927 but the cost model was not adjusted.
I believe the cost model was also incorrect for the old expansion.
The expansion prior to D47927 used 3 icmps using LHS, RHS, and Result
to calculate theirs signs. Then 2 icmps to compare the signs. Followed
by an And. The previous cost model was using 3 icmps and 2 selects.
Digging back through git blame, those 2 selects in the cost model used to
be 2 icmps, but were changed in https://reviews.llvm.org/D90681
Differential Revision: https://reviews.llvm.org/D110739
Diffstat (limited to 'llvm/lib/Analysis/BasicAliasAnalysis.cpp')
0 files changed, 0 insertions, 0 deletions