From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jan Hubicka To: Richard Kenner Cc: jh@suse.cz, gcc-patches@gcc.gnu.org Subject: Re: spec2000 regression Date: Wed, 15 Aug 2001 06:03:00 -0000 Message-id: <20010815150251.Y19872@atrey.karlin.mff.cuni.cz> References: <10108151302.AA17923@vlsi1.ultra.nyu.edu> X-SW-Source: 2001-08/msg00905.html > the problem is that actually code first sets the DECL_RTL and then > calls set_mem_attributes, that calls get_alias_set, but it thinks that > the alias set is already computed, but it isn't. > > I am now testing following patch. OK to install if it succeeds? > > I'd prefer to fix things so they don't set DECL_RTL first. MEM_ALIAS_SET > is not a test to see if the alias set has been set. The alias set might > have been zero in which case we *do* want to use it. That what I was trying to do first, but there are number of places this is happening and fixing each of it still keeps open door for possible new bugs - it is natural to save the RTL to proper place first. If I understand the purpose of original patch properly, it was hooting for case variable gets unique class, but this class is recomputed several times so it get several unique classes. For class 0 this does not matter. Honza > > So I think your change is wrong.