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: RFA: tuples merge for GIMPLE_MODIFY_STMT

Aldy Hernandez wrote on 11/11/06 04:47:

I have fixed this in the included patch, by generating a GIMPLE_MODIFY_STMT
instead of INIT_EXPR.  I'm not to gimplifier savvy enough to know if this will
miss out on optimizations, or if I'm side stepping some tenet of the
gimplifier.  This approach fixes the problem, bootstraps, and does not
introduce any regressions.

What do you think?

It shouldn't block anything. We don't really use INIT_EXPR for anything these days.


        * tree.c (build2_stat): Fix comment.
        * gimplify.c (internal_get_tmp_var): Change INIT_EXPR to
        GIMPLE_MODIFY_STMT.
        (tree_to_gimple_tuple): Remove debugging statement.
        Do not re-set the DEF_STMT link.

Yes, this looks fine.

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