This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH][ARM][testsuite] Fix testsuite options for testing rounding vectorisation on ARMv8
- From: Ramana Radhakrishnan <ramrad01 at arm dot com>
- To: Kyrylo Tkachov <Kyrylo dot Tkachov at arm dot com>
- Cc: "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>, "mikestump at comcast dot net" <mikestump at comcast dot net>
- Date: Fri, 05 Apr 2013 15:05:30 +0100
- Subject: Re: [PATCH][ARM][testsuite] Fix testsuite options for testing rounding vectorisation on ARMv8
- References: <01b101ce31fe$62857250$279056f0$ at tkachov@arm.com>
On 04/05/13 14:06, Kyrylo Tkachov wrote:
Hi all,
With r197491 I added testsuite support for vectorisation of rounding
functions on ARMv8 NEON, but the options set up
for vect.exp results in the testsuite trying to test all the vect tests with
ARMv8 NEON which does not work on
ARMv7 targets and simulators that don't support ARMv8 (like qemu).
But if we run the tests using v7 NEON, the newly enabled vect-rounding*
tests will FAIL because they need ARMv8 NEON options.
Therefore this patch reverts most of that and instead copies the rounding
vectorisation tests to gcc.target/arm
where the correct options can be set.
Tested arm-none-eabi on qemu to make sure that the execution tests come back
and use v7 NEON
Ok for trunk?
Ok by me but I'd like Mike to have another look.
It's a bit unfortunate we need to copy these tests over for the
architecture levels we need - there is a good project to restructure
gcc.target/arm to test properly at different arch. levels but for the
minute this is the best compromise.
Ramana
Thanks,
Kyrill
gcc/testsuite
2013-04-05 Kyrylo Tkachov <kyrylo.tkachov@arm.com>
* lib/target-supports.exp (add_options_for_arm_v8_neon):
Add -march=armv8-a when we use v8 NEON.
(check_effective_target_vect_call_btruncf): Remove arm-*-*-*.
(check_effective_target_vect_call_ceilf): Likewise.
(check_effective_target_vect_call_floorf): Likewise.
(check_effective_target_vect_call_roundf): Likewise.
(check_vect_support_and_set_flags): Remove check for arm_v8_neon.
* gcc.target/arm/vect-rounding-btruncf.c: New testcase.
* gcc.target/arm/vect-rounding-ceilf.c: Likewise.
* gcc.target/arm/vect-rounding-floorf.c: Likewise.
* gcc.target/arm/vect-rounding-roundf.c: Likewise.