diff options
| author | Sanjay Patel <spatel@rotateright.com> | 2018-11-05 16:50:44 +0000 |
|---|---|---|
| committer | Sanjay Patel <spatel@rotateright.com> | 2018-11-05 16:50:44 +0000 |
| commit | 87aa10062c0170a6a8cb79427655a086cf253b97 (patch) | |
| tree | dbfd48948ba83c1ead2f5c7164a36b1d5f36a582 /clang/lib/Format/Format.cpp | |
| parent | 8b2a1f7fd92a858d8d71a4bc9dcbd20f8af62dc4 (diff) | |
| download | bcm5719-llvm-87aa10062c0170a6a8cb79427655a086cf253b97.tar.gz bcm5719-llvm-87aa10062c0170a6a8cb79427655a086cf253b97.zip | |
[InstCombine] loosen FP 0.0 constraint for fcmp+select substitution
It looks like we correctly removed edge cases with 0.0 from D50714,
but we were a bit conservative because getBinOpIdentity() doesn't
distinguish between +0.0 and -0.0 and 'nsz' is effectively always
true for fcmp (see discussion in:
https://bugs.llvm.org/show_bug.cgi?id=38086
Without this change, we would get regressions by canonicalizing
to +0.0 in all fcmp, and that's a step towards solving:
https://bugs.llvm.org/show_bug.cgi?id=39475
llvm-svn: 346143
Diffstat (limited to 'clang/lib/Format/Format.cpp')
0 files changed, 0 insertions, 0 deletions
