On Wed, Jan 15, 2014 at 08:23:45AM -0800, Richard Henderson wrote:
> --- a/gcc/reginfo.c
> +++ b/gcc/reginfo.c
> @@ -86,7 +86,7 @@ static const char initial_call_really_used_regs[] = 
> CALL_REALLY_USED_REGISTERS;
>  char global_regs[FIRST_PSEUDO_REGISTER];
>  
>  /* Declaration for the global register. */
> -static tree GTY(()) global_regs_decl[FIRST_PSEUDO_REGISTER];
> +tree GTY(()) global_regs_decl[FIRST_PSEUDO_REGISTER];

Do you need the GTY(()) marker here when there is GTY(()) already in the
header?
Otherwise looks good to me.

>  /* Same information as REGS_INVALIDATED_BY_CALL but in regset form to be used
>     in dataflow more conveniently.  */
> diff --git a/gcc/rtl.h b/gcc/rtl.h
> index e7d60ee..10ee818 100644
> --- a/gcc/rtl.h
> +++ b/gcc/rtl.h
> @@ -2795,6 +2795,8 @@ extern void _fatal_insn (const char *, const_rtx, const 
> char *, int, const char
>  #define fatal_insn_not_found(insn) \
>       _fatal_insn_not_found (insn, __FILE__, __LINE__, __FUNCTION__)
>  
> +/* reginfo.c */
> +extern tree GTY(()) global_regs_decl[FIRST_PSEUDO_REGISTER];
>  
>  
>  #endif /* ! GCC_RTL_H */

        Jakub

Reply via email to