Ping.
https://gcc.gnu.org/ml/gcc-patches/2016-05/msg00842.html
Thanks,
Kyrill
On 11/05/16 15:32, Kyrill Tkachov wrote:
Hi all,
In this PR a NEON builtin is introduced during SLP vectorisation even when NEON
is not available
because arm_builtin_vectorized_function is missing an appropriate check in the
BSWAP handling code.
Then during expand when we try to expand the NEON builtin the code in
arm_expand_neon_builtin rightly
throws an error telling the user to enable NEON, even though the testcase
doesn't use any intrinsics.
This patch fixes the bug by bailing out early if !TARGET_NEON. This allows us
to remove a redundant
TARGET_NEON check further down in the function as well.
Bootstrapped and tested on arm-none-linux-gnueabihf.
Ok for trunk?
This appears on GCC 6 as well.
On older branches the test failure doesn't trigger but the logic looks buggy
anyway.
Ok for the branches as well if testing is clean?
Thanks,
Kyrill
2016-05-11 Kyrylo Tkachov <kyrylo.tkac...@arm.com>
PR target/71056
* config/arm/arm-builtins.c (arm_builtin_vectorized_function): Return
NULL_TREE early if NEON is not available. Remove now redundant check
in ARM_CHECK_BUILTIN_MODE.
2016-05-11 Kyrylo Tkachov <kyrylo.tkac...@arm.com>
PR target/71056
* gcc.target/arm/pr71056.c: New test.
