diff options
| author | Craig Topper <craig.topper@gmail.com> | 2020-01-02 20:51:13 -0800 |
|---|---|---|
| committer | Craig Topper <craig.topper@gmail.com> | 2020-01-02 21:46:53 -0800 |
| commit | bd46e29742759dd0d57feb0f33affa9adc7d2fbf (patch) | |
| tree | b34c4ba55f79d97ea71a3c1ee73d07c7a79e5069 /llvm/tools/llvm-size/llvm-size.cpp | |
| parent | 2133d3c5586b1a782e4d8e2a34c9f501499705cf (diff) | |
| download | llvm-bd46e29742759dd0d57feb0f33affa9adc7d2fbf.zip llvm-bd46e29742759dd0d57feb0f33affa9adc7d2fbf.tar.gz llvm-bd46e29742759dd0d57feb0f33affa9adc7d2fbf.tar.bz2 | |
[X86] Re-enable lowerUINT_TO_FP_vXi32 under fast-math by using an FSUB instead of an FADD.
Summary:
We previously disabled this under fast math due to aggressive
reassociation by the machine combiner. But I think we can work
around this by using a FSUB instead of FADD for the first
operation.
This matches the similar algorithm we do for uint_to_fp i64->f64
in TargetLowering::expandUINT_TO_FP. If reassociation hasn't
been a problem for that, hopefully its not a problem here.
Reviewers: RKSimon, spatel, scanon
Reviewed By: spatel
Subscribers: hiraditya, llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D71968
Diffstat (limited to 'llvm/tools/llvm-size/llvm-size.cpp')
0 files changed, 0 insertions, 0 deletions