Hi David,

On Mon, Oct 17, 2016 at 09:54:35AM -0400, David Edelsohn wrote:
> This patch shifts earlier the point during compilation where the
> rs6000 backend adds decoration to symbols: to encode_section_info.

> -  /* Currently C++ toc references to vtables can be emitted before it
> -     is decided whether the vtable is public or private.  If this is
> -     the case, then the linker will eventually complain that there is
> -     a reference to an unknown section.  Thus, for vtables only,
> -     we emit the TOC reference to reference the symbol and not the
> -     section.  */

I think this comment should go...

>    if (VTABLE_NAME_P (name))
>      {
>        RS6000_OUTPUT_BASENAME (file, name);

... before here.

Otherwise looks fine.  Do you want this tested on Linux before you
commit?  We never test on AIX beforehand either ;-)


Segher

Reply via email to