diff options
| author | Sanjay Patel <spatel@rotateright.com> | 2019-06-09 16:22:01 +0000 |
|---|---|---|
| committer | Sanjay Patel <spatel@rotateright.com> | 2019-06-09 16:22:01 +0000 |
| commit | 87cd16a86efbdfe0c8dd37645ac0d4bc5d6aeb86 (patch) | |
| tree | 4facd3cbef3d751f871fa623f7cc2e7a905e7516 /llvm/lib/Bitcode/Reader/BitcodeReader.cpp | |
| parent | 96ccd690f8eb85de7db6eb5274d882049e18b517 (diff) | |
| download | llvm-87cd16a86efbdfe0c8dd37645ac0d4bc5d6aeb86.zip llvm-87cd16a86efbdfe0c8dd37645ac0d4bc5d6aeb86.tar.gz llvm-87cd16a86efbdfe0c8dd37645ac0d4bc5d6aeb86.tar.bz2 | |
[InstCombine] change canonicalization to fabs() to use FMF on fneg
This isn't the ideal fix (use FMF on the select), but it's still an
improvement until we have better FMF propagation to selects and other
FP math operators.
I don't think there's much risk of regression from this change by
not including the FMF on the fcmp any more. The nsz/nnan FMF
should be the same on the fcmp and the fneg (fsub) because they
have the same operand.
This works around the most glaring FMF logical inconsistency cited
in PR38086:
https://bugs.llvm.org/show_bug.cgi?id=38086
llvm-svn: 362909
Diffstat (limited to 'llvm/lib/Bitcode/Reader/BitcodeReader.cpp')
0 files changed, 0 insertions, 0 deletions