From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 13393 invoked by alias); 13 Jul 2007 17:19:08 -0000 Received: (qmail 13336 invoked by alias); 13 Jul 2007 17:18:59 -0000 Date: Fri, 13 Jul 2007 17:19:00 -0000 Message-ID: <20070713171859.13335.qmail@sourceware.org> X-Bugzilla-Reason: CC References: Subject: [Bug tree-optimization/32705] [4.3 regression] ICE in set_ssa_val_to, at tree-ssa-sccvn.c:1022 In-Reply-To: Reply-To: gcc-bugzilla@gcc.gnu.org To: gcc-bugs@gcc.gnu.org From: "dberlin at dberlin dot org" Mailing-List: contact gcc-bugs-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Id: List-Archive: List-Post: List-Help: Sender: gcc-bugs-owner@gcc.gnu.org X-SW-Source: 2007-07/txt/msg01529.txt.bz2 ------- 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 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