On May 4, 2011, at 5:08 AM, Rainer Orth wrote: > The following patch is a prerequisite for making > > [lto, testsuite] Don't use visibility on targets that don't support it > (PR lto/47334) > http://gcc.gnu.org/ml/gcc-patches/2011-04/msg00295.html
> -# define USE_LINKONCE_INDIRECT (SUPPORTS_ONE_ONLY) > +# define USE_LINKONCE_INDIRECT (SUPPORTS_ONE_ONLY) && !TARGET_MACHO Generally speaking, we don't litter the backend with things like this. We consider this trashy, and we limit the trash to config/... Now, if you invent a feature (bug) for which this is really testing, and used it instead here, and then put that into the darwin.h file or into an as autoconf test, I think it would be fine. Ok with that version. If a build/configure/visibility person wants to object or insist on a better way to do what you want to do, I'd defer to them.