This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [tuples][patch] Removing gimplify_val
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.