This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [tuples][patch] Removing gimplify_val
Oh, thanks for the suggestion - I'll work on moving this into the
mainline as well.
Oleg
On Fri, Mar 14, 2008 at 7:24 AM, Richard Guenther
<richard.guenther@gmail.com> wrote:
>
> On Fri, Mar 14, 2008 at 3:20 PM, Diego Novillo <dnovillo@google.com> wrote:
> > On 3/14/08 2:37 AM, Richard Guenther wrote:
> >
> > > Can you do so on the trunk as well? The only difference I see with
> > > the two functions is that gimplify_val sets location information on
> > > the new stmt - you may want to do some digging in the archives
> > > for what reason this went in and eventually update force_gimple_operand_bsi
> > > to do so as well.
> >
> > Good idea. Oleg, please submit this patch for mainline. Make
> > force_gimple_operand_gsi set the location of all the statements in STMTS
> > in the loop that scans for symbols to rename.
>
> There's also the related cleanup (that was posted for 4.3 but taken out again -
> I don't remember by whom) of moving the symbol renaming stuff from
> force_gimple_operand_bsi to force_gimple_operand to have their
> behavior consistent in thsi regard.
>
> Richard.
>