This is the mail archive of the
gcc-bugs@gcc.gnu.org
mailing list for the GCC project.
[Bug tree-optimization/32705] [4.3 regression] ICE in set_ssa_val_to, at tree-ssa-sccvn.c:1022
- From: "dberlin at dberlin dot org" <gcc-bugzilla at gcc dot gnu dot org>
- To: gcc-bugs at gcc dot gnu dot org
- Date: 13 Jul 2007 17:18:59 -0000
- Subject: [Bug tree-optimization/32705] [4.3 regression] ICE in set_ssa_val_to, at tree-ssa-sccvn.c:1022
- References: <bug-32705-10053@http.gcc.gnu.org/bugzilla/>
- Reply-to: gcc-bugzilla at gcc dot gnu dot org
------- Comment #12 from dberlin at gcc dot gnu dot org 2007-07-13 17:18 -------
Subject: Re: [4.3 regression] ICE in set_ssa_val_to, at tree-ssa-sccvn.c:1022
On 13 Jul 2007 17:16:27 -0000, ebotcazou at gcc dot gnu dot org
<gcc-bugzilla@gcc.gnu.org> wrote:
>
>
> ------- Comment #11 from ebotcazou at gcc dot gnu dot org 2007-07-13 17:16 -------
> > Nah, that's not quite right, since this is a legal value.
> > Instead, where we init everything to VN_TOP, init everything with
> > SSA_NAME_OCCURS_IN_ABNORMAL_PHI to itself instead of VN_TOP.
>
> Note that we already deal with SSA_NAME_OCCURS_IN_ABNORMAL_PHI in visit_use,
> but not for PHI nodes:
>
> if (TREE_CODE (stmt) == PHI_NODE)
> {
> changed = visit_phi (stmt);
> }
> else if (TREE_CODE (stmt) != GIMPLE_MODIFY_STMT
> || (ann && ann->has_volatile_ops))
> {
> changed = defs_to_varying (stmt);
> }
>
> [...]
>
> if (TREE_CODE (lhs) == SSA_NAME
> && SSA_NAME_OCCURS_IN_ABNORMAL_PHI (lhs))
> changed = defs_to_varying (stmt);
>
> What about doing the same for them?
Sure, that would work too.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=32705