Re: [PATCH] Fix PR79278, amend ADJUST_FIELD_ALIGN interface

2017-02-01 Thread Segher Boessenkool
On Wed, Feb 01, 2017 at 03:53:09PM -0600, Segher Boessenkool wrote: > On Wed, Feb 01, 2017 at 11:59:10AM +0100, Richard Biener wrote: > > Wasn't successful in making a cross to ppc64-linux build its libobjc. > > I'll do a native build. Just the patch in the first message in this > thread? And ju

Re: [RFC] Bug lto/78140

2017-02-01 Thread kugan
Hi Richard, On 30/01/17 21:08, Richard Biener wrote: On Mon, Jan 30, 2017 at 12:23 AM, kugan wrote: Hi All, As suggested by Richard in the PR, I tried to implement variable size structures for VR as shown in attached patch. That is, I changed ipa-prop.h to: diff --git a/gcc/ipa-prop.h b/gcc/

Re: [PATCH doc] clean up -fdump-tree- options (PR 32003)

2017-02-01 Thread Sandra Loosemore
On 02/01/2017 06:57 PM, Martin Sebor wrote: As discussed in bug 32003 - Undocumented -fdump-tree options, rather than duplicating the same boiler-plate text for each of the dozens (138 by my count) of undocumented passes, the attached patch removes the pass-specific -fdump-tree- options replacing

Re: [PATCH doc] clean up -fdump-tree- options (PR 32003)

2017-02-01 Thread Martin Sebor
On 02/01/2017 08:06 PM, Sandra Loosemore wrote: On 02/01/2017 06:57 PM, Martin Sebor wrote: As discussed in bug 32003 - Undocumented -fdump-tree options, rather than duplicating the same boiler-plate text for each of the dozens (138 by my count) of undocumented passes, the attached patch removes

Re: [PATCH] [AArch64] Implement popcount pattern

2017-02-01 Thread Hurugalawadi, Naveen
Hi James and Kyrill, Thanks for the review and comments on the patch. >> On ILP32 systems this would still use the SImode patterns, >> so I suggest you use __builtin_popcountll and >> an unsigned long long return type to ensure you always exercise the 64-bit >> code. Sorry for not commenting o

Re: [PATCH] [AArch64] Enable AES and cmp_branch fusion for Thunderx2t99

2017-02-01 Thread Hurugalawadi, Naveen
Hi Kyrill, Thanks for the review and comments. >> but there are a couple of issues with the ChangeLog 2017-02-02 Naveen H.S * config/aarch64/aarch64.c (thunderx2t99_tunings): Enable AES and cmp_branch fusion. Thanks, Naveen

Re: [PATCH/AARCH64] Add scheduler for Thunderx2t99

2017-02-01 Thread Hurugalawadi, Naveen
Hi James, Thanks for reviewing the patch and comments. >> I wonder whether the current modeling of: >> (define_insn_reservation "thunderx2t99_asimd_load4_elts" 6 >> Actually benefits the schedule in a meaningful way, or if it just increases Done. Removed the scheduler modeling for thunderx2t99_a

Re: [PATCH doc] clean up -fdump-tree- options (PR 32003)

2017-02-01 Thread Sandra Loosemore
On 02/01/2017 08:26 PM, Martin Sebor wrote: On 02/01/2017 08:06 PM, Sandra Loosemore wrote: On 02/01/2017 06:57 PM, Martin Sebor wrote: As discussed in bug 32003 - Undocumented -fdump-tree options, rather than duplicating the same boiler-plate text for each of the dozens (138 by my count) of un

Re: [PATCH] Fix ICEs with power8 fixuns_truncdi2 patterns (PR target/79197)

2017-02-01 Thread Segher Boessenkool
On Mon, Jan 30, 2017 at 10:27:15PM +0100, Jakub Jelinek wrote: > Accoring to make mddump generated tmp-mddump.md, on powerpc the only pattern > with unsigned_fix:DI where the inner operand is SF or DFmode is the > *fixuns_truncdi2_fctiduz. There is an expander for that instruction, > which uses di

Re: [PATCH] Fix ICEs with power8 fixuns_truncdi2 patterns (PR target/79197)

2017-02-01 Thread Segher Boessenkool
On Tue, Jan 31, 2017 at 09:11:11AM +0100, Jakub Jelinek wrote: > What I think this patch doesn't handle properly is SFmode -> DImode > unsigned_fix. I have no idea how SFmode is represented in vector registers, > so have no idea if xscvdpuxds actually handles SFmode as well. All floating point sc

Re: [PATCH, pr63256] update powerpc dg options

2017-02-01 Thread Segher Boessenkool
On Thu, Jan 19, 2017 at 05:00:18PM -0600, Aaron Sawdey wrote: > SMS does process the loop in sms-8.c on powerpc now so I have updated > the options to reflect that. It passes with the --param, yes. Okay for trunk. Thanks, Segher

Re: [PATCH] use zero as the lower bound for a signed-unsigned range (PR 79327)

2017-02-01 Thread Jakub Jelinek
On Wed, Feb 01, 2017 at 04:52:35PM -0700, Martin Sebor wrote: > --- a/gcc/gimple-ssa-sprintf.c > +++ b/gcc/gimple-ssa-sprintf.c > @@ -1382,13 +1382,26 @@ format_integer (const directive &dir, tree arg) > would reflect the largest possible precision (i.e., INT_MAX). */ >res.range.m

Re: [PATCH] Fix ICEs with power8 fixuns_truncdi2 patterns (PR target/79197)

2017-02-01 Thread Jakub Jelinek
On Wed, Feb 01, 2017 at 11:56:34PM -0600, Segher Boessenkool wrote: > On Mon, Jan 30, 2017 at 10:27:15PM +0100, Jakub Jelinek wrote: > > Accoring to make mddump generated tmp-mddump.md, on powerpc the only pattern > > with unsigned_fix:DI where the inner operand is SF or DFmode is the > > *fixuns_t

<    1   2