> Hi, > > On Tue, Oct 23, 2012 at 3:03 AM, Jan Hubicka <hubi...@ucw.cz> wrote: > >> Ping. > >> > >> > >> On Wed, Oct 17, 2012 at 1:48 PM, Easwaran Raman <era...@google.com> wrote: > >> > Hi, > >> > This patch fixes bugs introduced by my previous patch to propagate > >> > profiles during switch expansion. Bootstrap and profiledbootstrap > >> > successful on x86_64. Confirmed that it fixes the crashes reported in > >> > PR middle-end/54957. OK for trunk? > >> > > >> > - Easwaran > >> > > >> > 2012-10-17 Easwaran Raman <era...@google.com> > >> > > >> > PR target/54938 > >> > PR middle-end/54957 > >> > * optabs.c (emit_cmp_and_jump_insn_1): Add REG_BR_PROB note > >> > only if it doesn't already exist. > >> > * except.c (sjlj_emit_function_enter): Remove unused variable. > >> > * stmt.c (get_outgoing_edge_probs): Return 0 if BB is NULL. > > > > Seems fine, but under what conditions you get NULL here? > Wasn't sure if this is an OK for the patch or if I need to address > anything else.
Actually I think you should make the except.c to setcurrent_bb when expanding the switch instead. OK with this change. Honza
