Re: PPC64 HTM support (was [buildbot] r201513: Invalid cast)

2013-08-07 Thread Peter Bergner
On Tue, 2013-08-06 at 22:01 -0500, Peter Bergner wrote: I'll bootstrap and regtest the change below and commit it as an obvious fix when that is done. Everything looked as as expected. Committed. Peter

Re: PPC64 HTM support (was [buildbot] r201513: Invalid cast)

2013-08-07 Thread Oleg Endo
On Tue, 2013-08-06 at 22:01 -0500, Peter Bergner wrote: Oleg Endo wrote: Speaking of GEN_FCN usage in rs6000.c. The recently added HTM builtin code has one interesting piece: static rtx htm_expand_builtin (tree exp, rtx target, bool * expandedp) { ... switch (nopnds) {

[buildbot] r201513: Invalid cast

2013-08-06 Thread Jan-Benedict Glaw
Hi! On Mon, 2013-08-05 22:09:45 -, olege...@gcc.gnu.org olege...@gcc.gnu.org wrote: Author: olegendo Date: Mon Aug 5 22:09:45 2013 New Revision: 201513 URL: http://gcc.gnu.org/viewcvs?rev=201513root=gccview=rev Log: PR other/12081 * recog.h (rtx (*insn_gen_fn) (rtx,

Re: [buildbot] r201513: Invalid cast

2013-08-06 Thread Oleg Endo
On Tue, 2013-08-06 at 13:27 +0200, Jan-Benedict Glaw wrote: Hi! On Mon, 2013-08-05 22:09:45 -, olege...@gcc.gnu.org olege...@gcc.gnu.org wrote: Author: olegendo Date: Mon Aug 5 22:09:45 2013 New Revision: 201513 URL: http://gcc.gnu.org/viewcvs?rev=201513root=gccview=rev

Re: [buildbot] r201513: Invalid cast

2013-08-06 Thread Richard Henderson
On 08/06/2013 11:20 AM, Oleg Endo wrote: PR other/12081 config/rs6000/rs6000.c (gen_2arg_fn_t): Remove typedef. (rs6000_emit_swdiv, rs6000_emit_swrsqrt): Don't cast result of GEN_FCN to gen_2arg_fn_t. Ok. r~

Re: PPC64 HTM support (was [buildbot] r201513: Invalid cast)

2013-08-06 Thread David Edelsohn
Peter, Would you please help answer Oleg questions about case 0 in htm_expand_builtin in rs6000.c? Thanks, David Speaking of GEN_FCN usage in rs6000.c. The recently added HTM builtin code has one interesting piece: static rtx htm_expand_builtin (tree exp, rtx target, bool * expandedp) { ...

Re: PPC64 HTM support (was [buildbot] r201513: Invalid cast)

2013-08-06 Thread Peter Bergner
Oleg Endo wrote: Speaking of GEN_FCN usage in rs6000.c. The recently added HTM builtin code has one interesting piece: static rtx htm_expand_builtin (tree exp, rtx target, bool * expandedp) { ... switch (nopnds) { case 0: pat = GEN_FCN (icode) (NULL_RTX); break; case 1: