This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH] Fix for ipa/63795, ipa/63622


> On Tue, Nov 11, 2014 at 3:22 PM, Martin Liška <mliska@suse.cz> wrote:
> > Hello.
> >
> > Following patch adds checking for aliasing support. Patch can bootstrap on
> > x86_64-apple-darwin1 and is part of patches needed for bootstrap restory on
> > the target. I plan to introduce additional patch that will cover testsuite
> > failures for the target.
> >
> > Ready for trunk?
> 
> "Aliasing" sounds odd here.  I'd expand it to "Symbol aliases", likewise
> rename target_supports_aliasing_p to target_supports_symbol_aliases_p.

I think it would move those predicates to symtab.  Also please add parameter
whether alias is weak.  Darwin supports aliases but no weak aliases, right?

Honza
> 
> Ok with that change.
> 
> Thanks,
> Richard.
> 
> > Thanks,
> > Martin


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]