This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [ARM] ARM NEON support part 1/7: VFPv3 support
- From: Julian Brown <julian at codesourcery dot com>
- To: Julian Brown <julian at codesourcery dot com>
- Cc: Richard Earnshaw <rearnsha at arm dot com>, GCC Patches <gcc-patches at gcc dot gnu dot org>, Paul Brook <paul at codesourcery dot com>
- Date: Wed, 04 Jul 2007 23:00:47 +0100
- Subject: Re: [ARM] ARM NEON support part 1/7: VFPv3 support
- References: <4661DE3A.20307@codesourcery.com> <1181236362.21501.12.camel@pc960.cambridge.arm.com> <468AA805.2040301@codesourcery.com>
Julian Brown wrote:
I've committed this version (r126272), which hopefully addresses all the
points mentioned (also a couple of Neon-specific bits which crept in
have been removed).
Richard E told me on IRC that this patch fails native bootstrap on ARM.
Richard, could you see if the attached (untested) patch fixes it for you
please?
OK to commit if successful?
Cheers,
Julian
ChangeLog
gcc/
* config/arm/arm.c (vfp3_const_double_index): Cast pointers passed
to REAL_VALUE_TO_INT to HOST_WIDE_INT*.
Index: arm.c
===================================================================
--- arm.c (revision 126316)
+++ arm.c (working copy)
@@ -5699,7 +5699,8 @@ vfp3_const_double_index (rtx x)
WARNING: If there's ever a VFP version which uses more than 2 * H_W_I - 1
bits for the mantissa, this may fail (low bits would be lost). */
real_ldexp (&m, &r, point_pos - exponent);
- REAL_VALUE_TO_INT (&mantissa, &mant_hi, m);
+ REAL_VALUE_TO_INT ((HOST_WIDE_INT *) &mantissa, (HOST_WIDE_INT *) &mant_hi,
+ m);
/* If there are bits set in the low part of the mantissa, we can't
represent this value. */