> With the patch, GCC on AIX now responds that "ifunc" is supported.
> The C and C++ attr-ifunc testcases now run and fail.

This is unexpected.  I am testing version of the patch with

  if (lookup_attribute ("ifunc", DECL_ATTRIBUTES (n->decl)))
    return false;

added to beggining of rs6000_declare_alias.  The problem here is that ifunc is
dealt with as an alias and becuase we output aliases ourselves, we skip the
sanity checking done in do_assemble_alias.  I think that checking is way too
late, will look into moving it earlier.

I also checked that we won't skip other useful sanity checking there. (other 
one is
weakref that we already know to not output in the new way)

Honza
> 
> g++.dg/ipa/devirt-10 and devirt-15 now fail.
> 
> Otherwise, the results look very good.
> 
> Thanks, David

Reply via email to