Re: [tuples][patch] Removing gimplify_val

["Oleg Ryjkov" <olegr@google.com>]

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.
>