This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [Patch ARM] Fixup testsuite for gcc.target/neon-modes-1.c
- From: Rainer Orth <ro at CeBiTec dot Uni-Bielefeld dot DE>
- To: "Joseph S. Myers" <joseph at codesourcery dot com>
- Cc: Ramana Radhakrishnan <ramana dot radhakrishnan at arm dot com>, gcc-patches <gcc-patches at gcc dot gnu dot org>, Richard Earnshaw <Richard dot Earnshaw at arm dot com>, froydnj at codesourcery dot com, Daniel Jacobowitz <dan at codesourcery dot com>
- Date: Wed, 07 Apr 2010 16:55:26 +0200
- Subject: Re: [Patch ARM] Fixup testsuite for gcc.target/neon-modes-1.c
- References: <1270564216.26267.68.camel@e200593-lin.cambridge.arm.com> <Pine.LNX.4.64.1004071444090.6130@digraph.polyomino.org.uk>
"Joseph S. Myers" <joseph@codesourcery.com> writes:
>> It appears as though this commit omitted adding the procedure
>> add_options_for_arm_neon to the testsuite or documenting it.
>
> No, add_options_for_arm_neon is actually part of a patch by Daniel
> Jacobowitz to improve testsuite handling of testing with -mfloat-abi,
> -mfpu etc. options (and is not supposed to add -mfpu=neon
> -mfloat-abi=softfp if the multilib options already imply those, or
> variants such as -mfpu=neon-fp16 or -mfloat-abi=hard). It's that patch
> which needs contributing.
When this is done, please document arm_neon in the appropriate section
of sourcebuild.texi. When revising the testsuite docs, I've already
noticed the missing procedure.
Thanks.
Rainer
--
-----------------------------------------------------------------------------
Rainer Orth, Center for Biotechnology, Bielefeld University