From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 24038 invoked by alias); 29 Aug 2007 11:47:09 -0000 Received: (qmail 23986 invoked by alias); 29 Aug 2007 11:47:09 -0000 Date: Wed, 29 Aug 2007 11:47:00 -0000 Message-ID: <20070829114709.23985.qmail@sourceware.org> X-Bugzilla-Reason: CC References: Subject: [Bug middle-end/32758] [4.3 Regression] ecj1 hangs In-Reply-To: Reply-To: gcc-bugzilla@gcc.gnu.org To: java-prs@gcc.gnu.org From: "paolo dot bonzini at lu dot unisi dot ch" Mailing-List: contact java-prs-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: java-prs-owner@gcc.gnu.org X-SW-Source: 2007-q3/txt/msg00139.txt.bz2 ------- Comment #23 from paolo dot bonzini at lu dot unisi dot ch 2007-08-29 11:47 ------- Subject: Re: [4.3 Regression] ecj1 hangs > df_simulate_one_insn_forwards and df_simulate_one_insn_backwards > (why we have the former when nothing ever uses it?) both call > df_simulate_fixup_sets to fix this up, shouldn't dce_process_block call that > too? Yes, it was probably an oversight of this patch: 2007-05-21 Kenneth Zadeck * dbgcnt.def: Fixed comment. * df-scan.c (df_get_regular_block_artificial_uses): Added frame pointer after reload if frame_pointer_needed. * df.h (df_simulate_defs, df_simulate_uses): Made public. * df-problems.c (df_simulate_defs, df_simulate_uses): Made public. * dce.c (deletable_insn_p): Only allow frame-related insns to be deleted if there is a REG_MAYBE_DEAD note. (dce_process_block): Now uses df_simulate_defs and df_simulate_uses. It should be placed outside the if, i.e. you should always execute it. You can CC me when you send the patch to the mailing list (but I'll be out of office from tomorrow to sunday). -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=32758