diff options
author | Matthias Braun <matze@braunis.de> | 2015-11-12 01:02:47 +0000 |
---|---|---|
committer | Matthias Braun <matze@braunis.de> | 2015-11-12 01:02:47 +0000 |
commit | b9610a6bc27530e5019c9a3181577b9d60ee1f32 (patch) | |
tree | 75fde15ba7fb72cd651a77dd9d93089ceba9c70f /clang/lib/CodeGen/CodeGenFunction.cpp | |
parent | dc3135db0593a6048f3373e061158022f9a6b283 (diff) | |
download | bcm5719-llvm-b9610a6bc27530e5019c9a3181577b9d60ee1f32.tar.gz bcm5719-llvm-b9610a6bc27530e5019c9a3181577b9d60ee1f32.zip |
LegalizeDAG: Fix and improve FCOPYSIGN/FABS legalization
- Factor out code to query and modify the sign bit of a floatingpoint
value as an integer. This also works if none of the targets integer
types is big enough to hold all bits of the floatingpoint value.
- Legalize FABS(x) as FCOPYSIGN(x, 0.0) if FCOPYSIGN is available,
otherwise perform bit manipulation on the sign bit. The previous code
used "x >u 0 ? x : -x" which is incorrect for x being -0.0! It also
takes 34 instructions on ARM Cortex-M4. With this patch we only
require 5:
vldr d0, LCPI0_0
vmov r2, r3, d0
lsrs r2, r3, #31
bfi r1, r2, #31, #1
bx lr
(This could be further improved if the compiler would recognize that
r2, r3 is zero).
- Only lower FCOPYSIGN(x, y) = sign(x) ? -FABS(x) : FABS(x) if FABS is
available otherwise perform bit manipulation on the sign bit.
- Perform the sign(x) test by masking out the sign bit and comparing
with 0 rather than shifting the sign bit to the highest position and
testing for "<s 0". For x86 copysignl (on 80bit values) this gets us:
testl $32768, %eax
rather than:
shlq $48, %rax
sets %al
testb %al, %al
Differential Revision: http://reviews.llvm.org/D11172
llvm-svn: 252839
Diffstat (limited to 'clang/lib/CodeGen/CodeGenFunction.cpp')
0 files changed, 0 insertions, 0 deletions