This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [ipa branch] SSA IPCP cleanups
- From: Jan Hubicka <hubicka at ucw dot cz>
- To: Razya Ladelsky <RAZYA at il dot ibm dot com>
- Cc: gcc-patches at gcc dot gnu dot org, Jan Hubicka <jh at suse dot cz>, hubicka at ucw dot cz
- Date: Tue, 31 Oct 2006 16:27:48 +0100
- Subject: Re: [ipa branch] SSA IPCP cleanups
- References: <OF4CFD8119.32D8F3AC-ONC2257218.00335C35-C2257218.0033CFBB@il.ibm.com>
> Hi,
>
> Attached are some cleanups to IPCP code on ipa branch (SSA based IPCP)
>
> Bootstrapped/regtested with ipa-cp enabled on ppc-linux
> OK for ipa branch?
>
> 2006-10-31 Razya Ladklesky <razya@il.ibm.com>
>
> * ipa-cp.c: Several cleanups.
- init_stmt =
- build2 (MODIFY_EXPR, void_type_node,
- new_ssa /*default_def (parm1) */ , val);
- TREE_OPERAND (init_stmt, 0) = new_ssa /*default_def (parm1) */ ;
- SSA_NAME_DEF_STMT (new_ssa /*default_def (parm1) */ ) = init_stmt;
+ init_stmt = build2 (MODIFY_EXPR, void_type_node, new_ssa, val);
+ TREE_OPERAND (init_stmt, 0) = new_ssa;
This should be already done by build2 call above.
+ if (init_stmt)
+ if (ENTRY_BLOCK_PTR_FOR_FUNCTION (func)->succs)
This if should be redundant as FOR_EACH_EDGE should not iterate then.
Otherwise it seems fine.
Honza