This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH PR79868 ][aarch64] Fix error calls in aarch64 code so they can be translated (version 2)
- From: Steve Ellcey <sellcey at cavium dot com>
- To: gcc-patches <gcc-patches at gcc dot gnu dot org>, Martin Sebor <msebor at gmail dot com>, fmarchal <fmarchal at perso dot be>, "roland.illig" <roland dot illig at gmx dot de>
- Date: Fri, 06 Oct 2017 15:23:38 -0700
- Subject: Re: [PATCH PR79868 ][aarch64] Fix error calls in aarch64 code so they can be translated (version 2)
- Authentication-results: sourceware.org; auth=none
- Authentication-results: spf=none (sender IP is ) smtp.mailfrom=Steve dot Ellcey at cavium dot com;
- References: <1506381921.18449.22.camel@cavium.com>
- Reply-to: sellcey at cavium dot com
- Spamdiagnosticmetadata: NSPM
- Spamdiagnosticoutput: 1:99
Ping.
Steve Ellcey
sellcey@cavium.com
On Mon, 2017-09-25 at 16:25 -0700, Steve Ellcey wrote:
> This is a new version of my patch to fix PR target/79868, where some
> error messages are impossible to translate correctly due to how the
> strings are dynamically constructed. It also includes some format
> changes in the error messags to make the messages more consistent with
> each other and with other GCC errors. This was worked out with help
> from Martin Sebor. I also had to fix some tests to match the new error
> string formats.
>
> Tested on Aarch64 with no regressions, OK to checkin?
>
> Steve Ellcey
> sellcey@cavium.com
>
>
> 2017-09-25 Steve Ellcey <sellcey@cavium.com>
>
> PR target/79868
> * config/aarch64/aarch64-c.c (aarch64_pragma_target_parse):
> Change argument type on aarch64_process_target_attr call.
> * config/aarch64/aarch64-protos.h
> (aarch64_process_target_attr):
> Change argument type.
> * config/aarch64/aarch64.c (aarch64_attribute_info): Change
> field type.
> (aarch64_handle_attr_arch): Change argument type, use boolean
> argument to use different strings in error calls.
> (aarch64_handle_attr_cpu): Ditto.
> (aarch64_handle_attr_tune): Ditto.
> (aarch64_handle_attr_isa_flags): Ditto.
> (aarch64_process_one_target_attr): Ditto.
> (aarch64_process_target_attr): Ditto.
> (aarch64_option_valid_attribute_p): Change argument type on
> aarch64_process_target_attr call.
>
>
> 2017-09-25 Steve Ellcey <sellcey@cavium.com>
>
> PR target/79868
> * gcc.target/aarch64/spellcheck_1.c: Update dg-error string to
> match
> new format.
> * gcc.target/aarch64/spellcheck_2.c: Ditto.
> * gcc.target/aarch64/spellcheck_3.c: Ditto.
> * gcc.target/aarch64/target_attr_11.c: Ditto.
> * gcc.target/aarch64/target_attr_12.c: Ditto.
> * gcc.target/aarch64/target_attr_17.c: Ditto.