This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [RFA:] fix breakage with "Update testsuite to run with slim LTO"
- From: Jan Hubicka <hubicka at ucw dot cz>
- To: Hans-Peter Nilsson <hans-peter dot nilsson at axis dot com>
- Cc: hubicka at ucw dot cz, hp at axis dot com, gcc-patches at gcc dot gnu dot org, dnovillo at google dot com, rguenther at suse dot de, ak at linux dot intel dot com
- Date: Fri, 21 Oct 2011 11:31:12 +0200
- Subject: Re: [RFA:] fix breakage with "Update testsuite to run with slim LTO"
- References: <20111020221932.GA13355@kam.mff.cuni.cz> <201110210229.p9L2TKmx010677@ignucius.se.axis.com>
> > Date: Fri, 21 Oct 2011 00:19:32 +0200
> > From: Jan Hubicka <hubicka@ucw.cz>
> > Yes, if we scan assembler, we likely want -fno-fat-lto-objects.
>
> > > then IIUC you need to patch *all* torture tests that use
> > > scan-assembler and scan-assembler-not. Alternatively, patch
> > > somewhere else, like not passing it if certain directives are
> > > used, like scan-assembler{,-not}. And either way, is it safe to
> > > add that option always, not just when also passing "-flto" or
> > > something?
> >
> > Hmm, some of assembler scans still works because they check for
> > presence of symbols we output anyway, but indeed, it would make more
> > sense to automatically imply -ffat-lto-object when scan-assembler
> > is used. I am not sure if my dejagnu skill as on par here however.
>
> Maybe you could make amends ;) by testing the following, which
> seems to work at least for dg-torture.exp and cris-elf/cris-sim,
> in which -ffat-lto-object is automatically added for each
> scan-assembler and scan-assembler-not test, extensible for other
> dg-final actions without polluting with checking LTO options and
> whatnot across the files. I checked (and corrected) so it also
> works when !check_effective_target_lto by commenting out the
> setting in the second chunk.
Thanks. It looks good to me. If we ever start scanning LTO assembler output,
we may simply add scan-lto-assembler variants or so...
Honza