diff options
| author | Sanjay Patel <spatel@rotateright.com> | 2016-06-02 20:01:37 +0000 |
|---|---|---|
| committer | Sanjay Patel <spatel@rotateright.com> | 2016-06-02 20:01:37 +0000 |
| commit | dba8b4c04d579b946572800d20419e70177b3194 (patch) | |
| tree | e5e683c188b16a381030fc74d498b32cee96c050 /llvm/lib/DebugInfo/PDB/PDBSymbolAnnotation.cpp | |
| parent | 617409f0c0a990d2d4b62f1a3e02ca26bc41d67a (diff) | |
| download | bcm5719-llvm-dba8b4c04d579b946572800d20419e70177b3194.tar.gz bcm5719-llvm-dba8b4c04d579b946572800d20419e70177b3194.zip | |
transform obscured FP sign bit ops into a fabs/fneg using TLI hook
This is effectively a revert of:
http://reviews.llvm.org/rL249702 - [InstCombine] transform masking off of an FP sign bit into a fabs() intrinsic call (PR24886)
and:
http://reviews.llvm.org/rL249701 - [ValueTracking] teach computeKnownBits that a fabs() clears sign bits
and a reimplementation as a DAG combine for targets that have IEEE754-compliant fabs/fneg instructions.
This is intended to resolve the objections raised on the dev list:
http://lists.llvm.org/pipermail/llvm-dev/2016-April/098154.html
and:
https://llvm.org/bugs/show_bug.cgi?id=24886#c4
In the interest of patch minimalism, I've only partly enabled AArch64. PowerPC, MIPS, x86 and others can enable later.
Differential Revision: http://reviews.llvm.org/D19391
llvm-svn: 271573
Diffstat (limited to 'llvm/lib/DebugInfo/PDB/PDBSymbolAnnotation.cpp')
0 files changed, 0 insertions, 0 deletions
