[tuples][patch] Removing gimplify_val

[Richard Guenther]

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.