[ARM] ARM NEON support part 1/7: VFPv3 support

Richard Earnshaw Richard.Earnshaw@buzzard.freeserve.co.uk
Thu Jul 5 19:34:00 GMT 2007


On Wed, 2007-07-04 at 23:00 +0100, Julian Brown wrote:
> 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*.

I can't say I like the casts on the pointers (ok, we're only changing
the signedness, but it's still dodgy in my book.  I think I'd prefer
something like the attached.  Agreed?

gcc/
	* config/arm/arm.c (vfp3_const_double_index): Copy signed 
	results of REAL_VALUE_TO_INT into unsigned vars.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: vfp.patch
Type: text/x-patch
Size: 1227 bytes
Desc: not available
URL: <http://gcc.gnu.org/pipermail/gcc-patches/attachments/20070705/8b96688e/attachment.bin>


More information about the Gcc-patches mailing list