This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCHv3][ARM] -mpure-code option for ARM
- From: "Andre Vieira (lists)" <Andre dot SimoesDiasVieira at arm dot com>
- To: Jakub Jelinek <jakub at redhat dot com>, Uros Bizjak <ubizjak at gmail dot com>
- Cc: "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>, Paolo Carlini <paolo dot carlini at oracle dot com>, Richard Earnshaw <Richard dot Earnshaw at arm dot com>
- Date: Fri, 23 Sep 2016 11:08:27 +0100
- Subject: Re: [PATCHv3][ARM] -mpure-code option for ARM
- Authentication-results: sourceware.org; auth=none
- References: <CAFULd4YCdAaD+iw_FFn=1b-0dtrX_WmMOGdJ=tsZKO_aA=z9+w@mail.gmail.com> <20160923100418.GM7282@tucnak.redhat.com>
On 23/09/16 11:04, Jakub Jelinek wrote:
> On Thu, Sep 22, 2016 at 10:37:21PM +0200, Uros Bizjak wrote:
>> diff --git a/gcc/hooks.c b/gcc/hooks.c
>> index 99ec4014adb6fcbb073bf538dd00fe8695ee6cb2..1e925645c3173f8d97e104b9b2f480fca2ede438
>> 100644
>> --- a/gcc/hooks.c
>> +++ b/gcc/hooks.c
>> @@ -481,3 +481,13 @@ void
>> hook_void_gcc_optionsp (struct gcc_options *opts ATTRIBUTE_UNUSED)
>> {
>> }
>> +
>> +/* Generic hook that takes an unsigned int, an unsigned int pointer and
>> + returns false. */
>> +
>> +bool
>> +hook_uint_uintp_false (unsigned int, unsigned int *)
>> +{
>> + return false;
>> +}
>>
>>
>> The name of this hook doesn't adhere to the convention. It should be named:
>>
>> hook_bool_uint_uintp_false
>
> You're right, I've committed this as obvious:
>
> 2016-09-23 Uros Bizjak <ubizjak@gmail.com>
> Jakub Jelinek <jakub@redhat.com>
>
> * hooks.h (hook_uint_uintp_false): Rename to...
> (hook_bool_uint_uintp_false): ... this.
> * hooks.c (hook_uint_uintp_false): Rename to...
> (hook_bool_uint_uintp_false): ... this.
> * target.def (elf_flags_numeric): Use hook_bool_uint_uintp_false
> instead of hook_uint_uintp_false.
>
> --- gcc/hooks.h.jj 2016-09-23 09:32:13.000000000 +0200
> +++ gcc/hooks.h 2016-09-23 11:57:28.116738504 +0200
> @@ -76,7 +76,7 @@ extern void hook_void_tree (tree);
> extern void hook_void_tree_treeptr (tree, tree *);
> extern void hook_void_int_int (int, int);
> extern void hook_void_gcc_optionsp (struct gcc_options *);
> -extern bool hook_uint_uintp_false (unsigned int, unsigned int *);
> +extern bool hook_bool_uint_uintp_false (unsigned int, unsigned int *);
>
> extern int hook_int_uint_mode_1 (unsigned int, machine_mode);
> extern int hook_int_const_tree_0 (const_tree);
> --- gcc/hooks.c.jj 2016-09-23 09:32:13.000000000 +0200
> +++ gcc/hooks.c 2016-09-23 11:57:39.938588268 +0200
> @@ -486,8 +486,7 @@ hook_void_gcc_optionsp (struct gcc_optio
> returns false. */
>
> bool
> -hook_uint_uintp_false (unsigned int, unsigned int *)
> +hook_bool_uint_uintp_false (unsigned int, unsigned int *)
> {
> return false;
> }
> -
> --- gcc/target.def.jj 2016-09-23 09:32:13.000000000 +0200
> +++ gcc/target.def 2016-09-23 11:59:03.581525303 +0200
> @@ -446,7 +446,7 @@ instead of the normal sequence of letter
> defined, or if it returns false, then @var{num} will be ignored and the\n\
> traditional letter sequence will be emitted.",
> bool, (unsigned int flags, unsigned int *num),
> - hook_uint_uintp_false)
> + hook_bool_uint_uintp_false)
>
> /* Return preferred text (sub)section for function DECL.
> Main purpose of this function is to separate cold, normal and hot
>
>
> Jakub
>
The hook that keeps on giving. Thanks! And one more time, sorry!
Cheers,
Andre