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) {

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: