This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

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.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]