On Tue, 1 Sep 2015, Martin Sebor wrote: > Attached is an updated patch that avoids diagnosing taking the address > of implicitly declared library builtins like abs, bootstrapped and > tested on ppc64le with no regressions. > > The tweak below was added to reject_gcc_builtin make it possible. > Since the expression is in c-family/c-common.c, the more descriptive > C_DECL_IMPLICIT that exists for this purpose is not available (it's > defined in c/c-tree.h). > > + && (c_dialect_cxx () || !DECL_LANG_FLAG_2 (expr))
I don't think hardcoding DECL_LANG_FLAG_2 here is a good idea; rather, some sort of c_decl_implicit hook should be defined (that would just return false for C++) - existing practice is various functions that have different definitions for C and C++. -- Joseph S. Myers jos...@codesourcery.com
