------- Comment #10 from dberlin at gcc dot gnu dot org 2007-07-13 16:47 ------- Subject: Re: [4.3 regression] ICE in set_ssa_val_to, at tree-ssa-sccvn.c:1022
On 13 Jul 2007 15:49:03 -0000, ebotcazou at gcc dot gnu dot org <[EMAIL PROTECTED]> wrote: > > > ------- Comment #9 from ebotcazou at gcc dot gnu dot org 2007-07-13 15:48 > ------- > And of course the naive patch: > > [EMAIL PROTECTED]:~/svn/gcc/gcc> Index: tree-ssa-sccvn.c > =================================================================== > --- tree-ssa-sccvn.c (revision 126547) > +++ tree-ssa-sccvn.c (working copy) > @@ -1279,7 +1279,7 @@ visit_phi (tree phi) > > /* If all value numbered to the same value, the phi node has that > value. */ > - if (allsame) > + if (allsame && sameval != VN_TOP) > { > if (is_gimple_min_invariant (sameval)) > { > 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. IE VN_INFO (a)->valnum = a iff SSA_NAME_OCCURS_IN_ABNORMAL_PHI (a) -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=32705