------- 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 <[EMAIL PROTECTED]>

         * 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

Reply via email to