Re: RFA: Store the REG_BR_PROB probability directly as an int

2013-09-27 Thread Eric Botcazou
> Thanks for the testing. It also passes bootstrap on x86_64-linux-gnu. > OK to install? Yes, thanks. -- Eric Botcazou

Re: RFA: Store the REG_BR_PROB probability directly as an int

2013-09-26 Thread Richard Sandiford
Steve Ellcey writes: > On Tue, 2013-09-24 at 21:07 +0200, Andreas Schwab wrote: >> Richard Sandiford writes: >> >> > Sorry for the breakage. I think we need to handle INT_LIST in the same way >> > as INSN_LIST though, and eliminate in XEXP (x, 1). >> > >> > How about the attached? Testing in p

Re: RFA: Store the REG_BR_PROB probability directly as an int

2013-09-25 Thread Steve Ellcey
On Tue, 2013-09-24 at 21:07 +0200, Andreas Schwab wrote: > Richard Sandiford writes: > > > Sorry for the breakage. I think we need to handle INT_LIST in the same way > > as INSN_LIST though, and eliminate in XEXP (x, 1). > > > > How about the attached? Testing in progress... > > Works for me a

Re: RFA: Store the REG_BR_PROB probability directly as an int

2013-09-24 Thread Andreas Schwab
Richard Sandiford writes: > Sorry for the breakage. I think we need to handle INT_LIST in the same way > as INSN_LIST though, and eliminate in XEXP (x, 1). > > How about the attached? Testing in progress... Works for me as well. Andreas. -- Andreas Schwab, sch...@linux-m68k.org GPG Key fing

Re: RFA: Store the REG_BR_PROB probability directly as an int

2013-09-24 Thread Richard Sandiford
Andreas Schwab writes: > Richard Sandiford writes: > >> REG_BR_PROB notes are stored as: >> >> (expr_list:REG_BR_PROB (const_int ) ) >> >> but a full const_int rtx seems a bit heavweight when all we want is >> a plain "int". This patch uses: >> >> (int_list:REG_BR_PROB ) >> >> instead. > >

Re: RFA: Store the REG_BR_PROB probability directly as an int

2013-09-24 Thread Andreas Schwab
Richard Sandiford writes: > REG_BR_PROB notes are stored as: > > (expr_list:REG_BR_PROB (const_int ) ) > > but a full const_int rtx seems a bit heavweight when all we want is > a plain "int". This patch uses: > > (int_list:REG_BR_PROB ) > > instead. I think you left out the handling of INT

Re: RFA: Store the REG_BR_PROB probability directly as an int

2013-09-23 Thread Richard Biener
On Sun, Sep 22, 2013 at 12:54 PM, Richard Sandiford wrote: > REG_BR_PROB notes are stored as: > > (expr_list:REG_BR_PROB (const_int ) ) > > but a full const_int rtx seems a bit heavweight when all we want is > a plain "int". This patch uses: > > (int_list:REG_BR_PROB ) > > instead. > > I did

Re: RFA: Store the REG_BR_PROB probability directly as an int

2013-09-22 Thread Jan Hubicka
> Hello Richard, > > Not directly related to your patch but... > > On Sun, Sep 22, 2013 at 12:54 PM, Richard Sandiford wrote: > > @@ -588,14 +589,17 @@ cond_exec_process_if_block (ce_if_block_ > > goto fail; > > #endif > > > > - true_prob_val = find_reg_note (BB_END (test_bb), REG_BR_PROB,

Re: RFA: Store the REG_BR_PROB probability directly as an int

2013-09-22 Thread Steven Bosscher
Hello Richard, Not directly related to your patch but... On Sun, Sep 22, 2013 at 12:54 PM, Richard Sandiford wrote: > @@ -588,14 +589,17 @@ cond_exec_process_if_block (ce_if_block_ > goto fail; > #endif > > - true_prob_val = find_reg_note (BB_END (test_bb), REG_BR_PROB, NULL_RTX); > - if

RFA: Store the REG_BR_PROB probability directly as an int

2013-09-22 Thread Richard Sandiford
REG_BR_PROB notes are stored as: (expr_list:REG_BR_PROB (const_int ) ) but a full const_int rtx seems a bit heavweight when all we want is a plain "int". This patch uses: (int_list:REG_BR_PROB ) instead. I didn't bother adding INT_LIST to lists.c because in practice we never free one exp