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