Patch ping (was Re: [PATCH] Don't ignore leading whitespace in AArch64 target attribute/pragma (PR target/89093))
Jakub Jelinek
jakub@redhat.com
Wed Apr 24 09:46:00 GMT 2019
Hi!
On Tue, Apr 16, 2019 at 08:32:50PM +0200, Jakub Jelinek wrote:
> 2019-04-16 Jakub Jelinek <jakub@redhat.com>
>
> PR target/89093
> * config/aarch64/aarch64.c (aarch64_process_one_target_attr): Don't skip
> whitespace at the start of target attribute string.
>
> * gcc.target/aarch64/pr89093.c: New test.
> * gcc.target/aarch64/pr63304_1.c: Remove space from target string.
I'd like to ping this patch.
Thanks.
> --- gcc/config/aarch64/aarch64.c.jj 2019-04-11 10:26:22.907293129 +0200
> +++ gcc/config/aarch64/aarch64.c 2019-04-15 19:59:55.784226278 +0200
> @@ -12536,10 +12536,6 @@ aarch64_process_one_target_attr (char *a
> char *str_to_check = (char *) alloca (len + 1);
> strcpy (str_to_check, arg_str);
>
> - /* Skip leading whitespace. */
> - while (*str_to_check == ' ' || *str_to_check == '\t')
> - str_to_check++;
> -
> /* We have something like __attribute__ ((target ("+fp+nosimd"))).
> It is easier to detect and handle it explicitly here rather than going
> through the machinery for the rest of the target attributes in this
> --- gcc/testsuite/gcc.target/aarch64/pr89093.c.jj 2019-04-15 20:02:25.456788897 +0200
> +++ gcc/testsuite/gcc.target/aarch64/pr89093.c 2019-04-15 20:02:04.433131260 +0200
> @@ -0,0 +1,7 @@
> +/* PR target/89093 */
> +/* { dg-do compile } */
> +
> +__attribute__((target (" no-strict-align"))) void f1 (void) {} /* { dg-error "is not valid" } */
> +__attribute__((target (" general-regs-only"))) void f2 (void) {} /* { dg-error "is not valid" } */
> +#pragma GCC target (" general-regs-only") /* { dg-error "is not valid" } */
> +void f3 (void) {}
> --- gcc/testsuite/gcc.target/aarch64/pr63304_1.c.jj 2017-09-13 16:22:19.795513580 +0200
> +++ gcc/testsuite/gcc.target/aarch64/pr63304_1.c 2019-04-15 20:27:17.724847578 +0200
> @@ -1,7 +1,7 @@
> /* { dg-do assemble } */
> /* { dg-options "-O1 --save-temps" } */
> #pragma GCC push_options
> -#pragma GCC target ("+nothing+simd, cmodel=small")
> +#pragma GCC target ("+nothing+simd,cmodel=small")
>
> int
> cal (double a)
Jakub
More information about the Gcc-patches
mailing list