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 sell...@mips.com writes: On Tue, 2013-09-24 at 21:07 +0200, Andreas Schwab wrote: Richard Sandiford rdsandif...@googlemail.com 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

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 rdsandif...@googlemail.com 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...

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

2013-09-24 Thread Andreas Schwab
Richard Sandiford rdsandif...@googlemail.com writes: REG_BR_PROB notes are stored as: (expr_list:REG_BR_PROB (const_int prob) chain) 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 prob chain) instead. I think

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

2013-09-24 Thread Richard Sandiford
Andreas Schwab sch...@suse.de writes: Richard Sandiford rdsandif...@googlemail.com writes: REG_BR_PROB notes are stored as: (expr_list:REG_BR_PROB (const_int prob) chain) but a full const_int rtx seems a bit heavweight when all we want is a plain int. This patch uses:

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

2013-09-24 Thread Andreas Schwab
Richard Sandiford rdsandif...@googlemail.com 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,

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 rdsandif...@googlemail.com wrote: REG_BR_PROB notes are stored as: (expr_list:REG_BR_PROB (const_int prob) chain) 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

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 prob) chain) 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 prob chain) instead. I didn't bother adding INT_LIST to lists.c because in practice we

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

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, NULL_RTX);