This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] PR35371 GCSE loses track of REG_POINTER attribute
- From: Peter Bergner <bergner at vnet dot ibm dot com>
- To: Jeff Law <law at redhat dot com>
- Cc: Richard Sandiford <rsandifo at nildram dot co dot uk>, gcc-patches <gcc-patches at gcc dot gnu dot org>, "H.J. Lu" <hjl at lucon dot org>
- Date: Mon, 03 Mar 2008 13:41:48 -0600
- Subject: Re: [PATCH] PR35371 GCSE loses track of REG_POINTER attribute
- References: <20080225222624.GA26857@vervain.rchland.ibm.com> <87skzfzkvh.fsf@firetop.home> <1204052325.7014.2.camel@otta> <47C467BA.1090308@redhat.com> <1204240130.7376.16.camel@otta> <47CC4E4D.5040706@redhat.com>
On Mon, 2008-03-03 at 12:15 -0700, Jeff Law wrote:
> > Do you mean fix it up and then call it from more than just CSE?
> > Currently, the only call to reg_scan() isn't in a location that
> > will help me.
> No. I mean make it smarter. If you read the code it's amazingly
> simplistic and punts propagation of REG_POINTER for any pseudo
> that is set more than once.
>
> It shouldn't be terribly difficult to build a simple propagation
> engine that handles multiple sets.
Sorry, making it "smarter" is what I meant by "fix it up".
My problem with it, as I mentioned in my previous note, is that
the only location it is currently called doesn't help me.
I guess what I was asking was there shouldn't be a problem
with me calling it from another location, correct?
H.J.,
Did you see my request to test the patch attached to:
http://gcc.gnu.org/ml/gcc-patches/2008-02/msg01442.html
to verify I haven't changed x86/x86_64's SPEC scores due to
the rtlanal.c change?
Peter