Re: [PATCH 0/8] make next_*_insn take rtx_insn * arguments

2016-09-19 Thread Trevor Saunders
On Mon, Sep 19, 2016 at 02:49:49PM -0600, Jeff Law wrote: > On 09/16/2016 04:10 AM, Trevor Saunders wrote: > > On Thu, Sep 15, 2016 at 10:27:56AM -0600, Jeff Law wrote: > > > On 09/15/2016 10:10 AM, Trevor Saunders wrote: > > > > On Thu, Sep 15, 2016 at 12:52:59PM +0200, Bernd Schmidt wrote: > > >

Re: [PATCH], Use VMRGEW on PowerPC power8/power9 to construct V4SFmode

2016-09-19 Thread Segher Boessenkool
On Mon, Sep 19, 2016 at 06:02:08PM -0400, Michael Meissner wrote: > vector float combine (float a, float b, float c, float d) > { > return (vector float) { a, b, c, d }; > } [ ... ] > However ISA 2.07 (i.e. power8) added the VMRGEW instruction, which can do this > more simply: > > xxpe

Re: [PATCH], Use VMRGEW on PowerPC power8/power9 to construct V4SFmode

2016-09-19 Thread Michael Meissner
On Mon, Sep 19, 2016 at 05:43:19PM -0500, Segher Boessenkool wrote: > On Mon, Sep 19, 2016 at 06:02:08PM -0400, Michael Meissner wrote: > > vector float combine (float a, float b, float c, float d) > > { > > return (vector float) { a, b, c, d }; > > } > > [ ... ] > > > However ISA 2.07 (i.e. po

Re: [PATCH 0/8] make next_*_insn take rtx_insn * arguments

2016-09-19 Thread Segher Boessenkool
On Mon, Sep 19, 2016 at 06:07:46PM -0400, Trevor Saunders wrote: > > > - JUMP_LABEL can be a return which is not an insn. That also seems > > > like something that should be improved at some point. > > The JUMP_LABEL field within a JUMP_INSN? That sounds like a bug. > > yes, see the comment be

Re: [PATCH 3/8] make next/prev _nonnote_insn take rtx_insn *

2016-09-19 Thread Trevor Saunders
On Mon, Sep 19, 2016 at 03:23:58PM -0600, Jeff Law wrote: > On 09/14/2016 01:21 PM, tbsaunde+...@tbsaunde.org wrote: > > From: Trevor Saunders > > > > gcc/ChangeLog: > > > > 2016-09-13 Trevor Saunders > > > > * emit-rtl.c (next_nonnote_insn): Change argument type to > > rtx_insn *. >

Re: [PATCH], Use VMRGEW on PowerPC power8/power9 to construct V4SFmode

2016-09-19 Thread Segher Boessenkool
On Mon, Sep 19, 2016 at 06:51:42PM -0400, Michael Meissner wrote: > > > However ISA 2.07 (i.e. power8) added the VMRGEW instruction, which can do > > > this > > > more simply: > > > > > > xxpermdi 34,1,2,0 > > > xxpermdi 32,3,4,0 > > > xvcvdpsp 34,34 > > > xvcvdpsp

Re: [PATCH 7/8] make next/prev active_insn and active_insn_p take rtx_insn *

2016-09-19 Thread Trevor Saunders
On Mon, Sep 19, 2016 at 03:17:33PM -0600, Jeff Law wrote: > On 09/14/2016 01:21 PM, tbsaunde+...@tbsaunde.org wrote: > > From: Trevor Saunders > > > > gcc/ChangeLog: > > > > 2016-09-14 Trevor Saunders > > > > * emit-rtl.c (next_active_insn): Change argument type to > > rtx_insn *. >

Re: [PATCH], Use VMRGEW on PowerPC power8/power9 to construct V4SFmode

2016-09-19 Thread Michael Meissner
On Mon, Sep 19, 2016 at 06:20:59PM -0500, Segher Boessenkool wrote: > On Mon, Sep 19, 2016 at 06:51:42PM -0400, Michael Meissner wrote: > > > > However ISA 2.07 (i.e. power8) added the VMRGEW instruction, which can > > > > do this > > > > more simply: > > > > > > > > xxpermdi 34,1,2,0 > >

Re: [PR72835] Incorrect arithmetic optimization involving bitfield arguments

2016-09-19 Thread kugan
Hi Richard, Thanks for the review. On 19/09/16 23:40, Richard Biener wrote: On Sun, Sep 18, 2016 at 10:21 PM, kugan wrote: Hi Richard, On 14/09/16 21:31, Richard Biener wrote: On Fri, Sep 2, 2016 at 10:09 AM, Kugan Vivekanandarajah wrote: Hi Richard, On 25 August 2016 at 22:24, Richard

Re: [RFC][IPA-VRP] Early VRP Implementation

2016-09-19 Thread kugan
Hi Richard, Thanks for the review. On 19/09/16 22:56, Richard Biener wrote: On Sun, Sep 18, 2016 at 10:50 PM, kugan wrote: Hi Richard, On 16/09/16 20:21, Richard Biener wrote: On Fri, Sep 16, 2016 at 7:59 AM, kugan wrote: Hi Richard, Thanks for the review. On 14/09/16 22:04, Richard

Re: PR35503 - warn for restrict pointer

2016-09-19 Thread Martin Sebor
and used pp_format for formatting arg_positions by adding specifier %I (name chosen arbitrarily). Does that look OK ? diff --git a/gcc/pretty-print.c b/gcc/pretty-print.c index a39815e..e8bd1ef 100644 --- a/gcc/pretty-print.c +++ b/gcc/pretty-print.c @@ -610,6 +610,23 @@ pp_format (pretty_printe

Re: [PATCH] manual: _Bool has trap representations

2016-09-19 Thread Florian Weimer
On 09/19/2016 11:26 PM, Joseph Myers wrote: On powerpc-darwin _Bool is 4-byte not 1-byte, so saying values are represented as bytes isn't accurate for all systems supported by GCC. Interesting. Is the treatment of 0/1/the rest still the same there? Thanks, Florian

<    1   2