This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: LTO partitioning reorg 4/n
- From: Jan Hubicka <hubicka at ucw dot cz>
- To: Andi Kleen <ak at linux dot intel dot com>
- Cc: Jan Hubicka <hubicka at ucw dot cz>, gcc-patches at gcc dot gnu dot org
- Date: Sat, 15 Sep 2012 20:56:55 +0200
- Subject: Re: LTO partitioning reorg 4/n
- References: <20120912163154.GA7130@kam.mff.cuni.cz> <20120915182320.GK4092@tassilo.jf.intel.com>
> > The patch also fixed undefined reference symbol seen with max partitioning on a testcase.
> > Here we fold through a reference to constant variable into a reference to constant pool.
> > Since initializer of the constant variable is not seen by the partitioner, we do not
> > duplicate the constant pool entry into the unit and we end up with undefined reference.
> > Fixed thus. I will try to prepare fix for 4.7, too. I think it is the problem
> > Andi is seeing with kernel build.
>
> I did quite some testing with this one with building kernels with
> different configurations (including over night randomconfig runs)
> and it seems to fix the previous problems. I reverted the workarounds
> (making random symbols visible) reverted. So looks good.
>
> Getting it into 4.7 would be appreciated too, otherwise I would need to
> keep the workarounds.
Good news, the 4.7 version of the patch will be bit different, I will try to do it
early next week. Thanks for testing!
Honza
>
> -Andi