From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 4988 invoked by alias); 25 May 2004 10:01:43 -0000 Mailing-List: contact gcc-bugs-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Archive: List-Post: List-Help: Sender: gcc-bugs-owner@gcc.gnu.org Received: (qmail 4947 invoked by alias); 25 May 2004 10:01:40 -0000 Date: Wed, 26 May 2004 10:43:00 -0000 Message-ID: <20040525100140.4946.qmail@sourceware.org> From: "rguenth at tat dot physik dot uni-tuebingen dot de" To: gcc-bugs@gcc.gnu.org In-Reply-To: <20040112101346.13653.rguenth@tat.physik.uni-tuebingen.de> References: <20040112101346.13653.rguenth@tat.physik.uni-tuebingen.de> Reply-To: gcc-bugzilla@gcc.gnu.org Subject: [Bug rtl-optimization/13653] [3.3 regression] -O2 -funroll-loop miscompiles POOMA testcase X-Bugzilla-Reason: CC X-SW-Source: 2004-05/txt/msg02869.txt.bz2 List-Id: ------- Additional Comments From rguenth at tat dot physik dot uni-tuebingen dot de 2004-05-25 10:01 ------- Subject: Re: [3.3 regression] -O2 -funroll-loop miscompiles POOMA testcase On Mon, 24 May 2004, ebotcazou at gcc dot gnu dot org wrote: > > ------- Additional Comments From ebotcazou at gcc dot gnu dot org 2004-05-24 16:01 ------- > Try first to revert this one: > > 2004-04-24 Richard Henderson > > PR bootstrap/14671 > * alias.c (alias_invariant, alias_invariant_size): Mark GTY. > (reg_known_value, reg_known_value_size): Likewise; make static. > (reg_known_equiv_p): Make static. > (clear_reg_alias_info): Update for new indexing. > (get_reg_known_value, set_reg_known_value): New. > (get_reg_known_equiv_p, set_reg_known_equiv_p): New. > (canon_rtx): Use them. > (init_alias_analysis): Likewise. Allocate reg_known_value with gc. > Don't play queer offsetting games with reg_known_value and > reg_known_equiv_p. > (end_alias_analysis): Don't free reg_known_value. > * rtl.h (get_reg_known_value, get_reg_known_equiv_p): Declare. > * sched-deps.c (reg_known_equiv_p, reg_known_value): Remove. > (deps_may_trap_p, sched_analyze_1, sched_analyze_2): Use the new > functions instead. Reverting the patch fixes the problems and testresults are then on-par with gcc 3.4! Very nice. Please revert the above change. Richard. -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=13653