diff options
| author | Sanjay Patel <spatel@rotateright.com> | 2015-09-23 22:28:18 +0000 |
|---|---|---|
| committer | Sanjay Patel <spatel@rotateright.com> | 2015-09-23 22:28:18 +0000 |
| commit | 13e8bbc23735453bf0cbdf5c7c07cb60419ed072 (patch) | |
| tree | 20a466c79cc63a03f21a579f5a8fb488fa2cdb6c /llvm/lib/Transforms/Utils/LoopUnroll.cpp | |
| parent | b457b68f8d2d5769127a15aacc6f6071ea4ebf57 (diff) | |
| download | llvm-13e8bbc23735453bf0cbdf5c7c07cb60419ed072.zip llvm-13e8bbc23735453bf0cbdf5c7c07cb60419ed072.tar.gz llvm-13e8bbc23735453bf0cbdf5c7c07cb60419ed072.tar.bz2 | |
set div/rem default values to 'expensive' in TargetTransformInfo's cost model
...because that's what the cost model was intended to do.
As discussed in D12882, this fix has a temporary unintended consequence for
SimplifyCFG: it causes us to not speculate an fdiv. However, two wrongs make
PR24818 right, and two wrongs make PR24343 act right even though it's really
still wrong.
I intend to correct SimplifyCFG and add to CodeGenPrepare to account for this
cost model change and preserve the righteousness for the bug report cases.
https://llvm.org/bugs/show_bug.cgi?id=24818
https://llvm.org/bugs/show_bug.cgi?id=24343
Differential Revision: http://reviews.llvm.org/D12882
llvm-svn: 248439
Diffstat (limited to 'llvm/lib/Transforms/Utils/LoopUnroll.cpp')
0 files changed, 0 insertions, 0 deletions