"Yangfei (Felix)" writes:
> Hi,
>
>> -Original Message-
>> From: Richard Sandiford [mailto:richard.sandif...@arm.com]
>> Sent: Wednesday, April 22, 2020 6:03 PM
>> To: Yangfei (Felix)
>> Cc: gcc-patches@gcc.gnu.org
>> Subject: Re: [PATC
Hi,
> -Original Message-
> From: Richard Sandiford [mailto:richard.sandif...@arm.com]
> Sent: Wednesday, April 22, 2020 6:03 PM
> To: Yangfei (Felix)
> Cc: gcc-patches@gcc.gnu.org
> Subject: Re: [PATCH PR94678] aarch64: unexpected result with -mgeneral-
> regs-onl
Mostly LGTM, just a couple of minor points:
"Yangfei (Felix)" writes:
> diff --git a/gcc/ChangeLog b/gcc/ChangeLog
> index 6e226cc..e956b69 100644
> --- a/gcc/ChangeLog
> +++ b/gcc/ChangeLog
> @@ -1,3 +1,23 @@
> +2020-04-22 Felix Yang
> +
> + PR target/94678
> + * config/aarch64/aarch6
> -Original Message-
> From: Richard Sandiford [mailto:richard.sandif...@arm.com]
> Sent: Tuesday, April 21, 2020 6:11 PM
> To: Yangfei (Felix)
> Cc: gcc-patches@gcc.gnu.org
> Subject: Re: [PATCH PR94678] aarch64: unexpected result with -mgeneral-
> regs-on
"Yangfei (Felix)" writes:
>> -Original Message-
>> From: Richard Sandiford [mailto:richard.sandif...@arm.com]
>> Sent: Tuesday, April 21, 2020 4:01 PM
>> To: Yangfei (Felix)
>> Cc: gcc-patches@gcc.gnu.org
>> Subject: Re: [PATCH PR94678] aarc
On Tue, Apr 21, 2020 at 08:37:09AM +, Yangfei (Felix) wrote:
> Yes, that's right. And I have a question here:
> Should aarch64_sve::init_builtins ()/aarch64_sve::handle_arm_sve_h () be
> guarded by TARGET_SVE?
>
> I mean in aarch64_init_builtins:
> if (TARGET_SVE)
> aarch64_sve::init_built
> -Original Message-
> From: Richard Sandiford [mailto:richard.sandif...@arm.com]
> Sent: Tuesday, April 21, 2020 4:01 PM
> To: Yangfei (Felix)
> Cc: gcc-patches@gcc.gnu.org
> Subject: Re: [PATCH PR94678] aarch64: unexpected result with -mgeneral-
> regs-only and sve
"Yangfei (Felix)" writes:
> Hi,
>
> It looks like there are several issues out there for sve codegen with
> -mgeneral-regs-only.
> I have created a bug for that:
> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94678
>
> We do ISA extension checks for SVE in
> check_required_extensions(aa
Hi,
It looks like there are several issues out there for sve codegen with
-mgeneral-regs-only.
I have created a bug for that:
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94678
We do ISA extension checks for SVE in
check_required_extensions(aarch64-sve-builtins.cc).
I think we may a