Message ID | 201009202330.44434.ebotcazou@adacore.com |
---|---|
State | New |
Headers | show |
On Mon, Sep 20, 2010 at 11:30 PM, Eric Botcazou <ebotcazou@adacore.com> wrote: > 2010-09-20 Eric Botcazou <ebotcazou@adacore.com> > > PR rtl-optimization/42775 > * cfgrtl.c (rest_of_pass_free_cfg): Recompute notes if delay slot > scheduling is enabled. This is in the "egad but what else" category. Qualifies as obvious IMHO. As far as I can tell from the reorg.c source, there are no other DF problems that reorg needs, that have to be recomputed before free'ing the CFG, right? Ciao! Steven
> As far as I can tell from the reorg.c source, there are no other DF > problems that reorg needs, that have to be recomputed before free'ing > the CFG, right? Yes, resource.c should be OK with LR + Notes.
Index: cfgrtl.c =================================================================== --- cfgrtl.c (revision 164451) +++ cfgrtl.c (working copy) @@ -421,7 +421,10 @@ rest_of_pass_free_cfg (void) /* The resource.c machinery uses DF but the CFG isn't guaranteed to be valid at that point so it would be too late to call df_analyze. */ if (optimize > 0 && flag_delayed_branch) - df_analyze (); + { + df_note_add_problem (); + df_analyze (); + } #endif free_bb_for_insn ();