Re: [RFA] Implement __VA_OPT__

> "Jason" == Jason Merrill writes: Jason> Do we want 1s for vaopt in the GNU rows, then? It seems to only be Jason> used for controlling the pedwarn about needing at least one argument Jason> for the variadic parameter. It seems reasonable to me, but I wasn't sure. I've

Re: [PATCH][GCC][ARM] Implement "arch" GCC pragma and "+" attributes [Patch (2/3)]

On 11/10/2017 03:56 AM, Tamar Christina wrote: Hi Sandra, I've respun the patch with the docs changes you requested. Regards, Tamar Thanks! The doc part is OK now. -Sandra

[patch][x86] -march=icelake

Hi, this patch adds new option -march=icelake. Isasets defined in: https://software.intel.com/sites/default/files/managed/c5/15/architecture-instruction-set-extensions-programming-reference.pdf I didn't add arch code to driver-i386.c, because there is no code available in SDM yet, only for

Re: VRP: x+1 and -x cannot be INT_MIN

On Sat, 11 Nov 2017, Jakub Jelinek wrote: On Sat, Nov 11, 2017 at 11:03:06PM +0100, Marc Glisse wrote: --- gcc/tree-vrp.c (revision 254629) +++ gcc/tree-vrp.c (working copy) @@ -2086,20 +2086,38 @@ extract_range_from_binary_expr_1 (value_ else

Re: [Patch, Fortran] PR 82932: [8 Regression] [OOP] ICE in update_compcall_arglist, at fortran/resolve.c:5837

On Sat, Nov 11, 2017 at 10:59:05PM +0100, Janus Weil wrote: > 2017-11-11 19:01 GMT+01:00 Steve Kargl : > > On Sat, Nov 11, 2017 at 01:01:59PM +0100, Janus Weil wrote: > >> > >> here is a close-to-trivial patch for an ICE-on-invalid problem. It > >> regtests

Re: VRP: x+1 and -x cannot be INT_MIN

On Sat, Nov 11, 2017 at 11:03:06PM +0100, Marc Glisse wrote: > --- gcc/tree-vrp.c(revision 254629) > +++ gcc/tree-vrp.c(working copy) > @@ -2086,20 +2086,38 @@ extract_range_from_binary_expr_1 (value_ >else > set_value_range_to_varying (vr); > >return; > } >

VRP: x+1 and -x cannot be INT_MIN

Hello, with undefined overflow, just because we know nothing about one of the arguments of an addition doesn't mean we can't say something about the result. We could constrain more the cases where we replace VR_VARYING with a full VR_RANGE, but I didn't want to duplicate too much logic. The

Re: [Patch, Fortran] PR 82932: [8 Regression] [OOP] ICE in update_compcall_arglist, at fortran/resolve.c:5837

2017-11-11 19:01 GMT+01:00 Steve Kargl : > On Sat, Nov 11, 2017 at 01:01:59PM +0100, Janus Weil wrote: >> >> here is a close-to-trivial patch for an ICE-on-invalid problem. It >> regtests cleanly. >> >> Ok for trunk? (If there are no objections, I'd like to

RE: [x86][patch] Add -march=cannonlake.

Hi Uros, I fixed comments. Btw, I haven't found skylake-avx512 in driver-i386.c at all. Is it intended or should I add it? Thanks, Julia gcc/ * config.gcc: Add -march=cannonlake. * config/i386/driver-i386.c (host_detect_local_cpu): Detect cannonlake. *

Tweak vector::_M_realloc_insert for code size

Hello, operator new can clobber memory, it is hard to teach the compiler otherwise since it is replaceable. Here I cache a couple values before the call to the allocator. I checked the result on this simple example: #include void f(std::vector){ v.push_back(0); } The patch does not affect

Re: [PATCH 2/4] enhance overflow and truncation detection in strncpy and strncat (PR 81117)

On 11/10/2017 03:52 PM, Marc Glisse wrote: On Sun, 6 Aug 2017, Martin Sebor wrote: +@item nonstring (@var{nonstring}) Hello, what is the "(@var{nonstring})" for? This attribute does not seem to take any argument... It's a copy and paste typo. I removed it in r254659. Thanks for pointing

Re: [Patch, Fortran] PR 82932: [8 Regression] [OOP] ICE in update_compcall_arglist, at fortran/resolve.c:5837

On Sat, Nov 11, 2017 at 01:01:59PM +0100, Janus Weil wrote: > > here is a close-to-trivial patch for an ICE-on-invalid problem. It > regtests cleanly. > > Ok for trunk? (If there are no objections, I'd like to commit this in > the coming days.) > > Cheers, > Janus > > 2017-11-11 Janus Weil

[committed] fix -Wstringop-truncation in powerpc64le bootstrap (PR 82948)

powerpc64le-linux bootstrap exposed an instance of the new -Wstringop-truncation warning for a use of strncpy that doesn't come up on x86_64. I committed r254658 as an obvious solution to unblock the bootstrap. Martin

Re: isl scheduler and spatial locality (Re: [PATCH][GRAPHITE] More TLC)

On Sun, Oct 01, 2017 at 11:58:30AM +0200, Sven Verdoolaege wrote: > For the approach pluto is taking, you'll have to look at the source > code, see pluto_intra_tile_optimize_band. > For the other two approaches I mentioned above, reports will > be made available within the next couple of weeks.

Remove unused variable in tree-ssa-loop-im.c

Hi, this is another artifcat of my conversion. freq_sum is now unused (but not reported because of += which we possibly could also warn for I guess) Bootstrapped/regtested x86_64-linux, comitted. * tree-ssa-loop-im.c (execute_sm_if_changed): Do not compute freq_sum. Index:

Fix maybe_hot_frequency_p

Hi, while converting maybe_hot_frequency_p to profile_counts I did not notice that it has its own function argument. This patch prevents ICE with -fdump-tree-ipa-all-details-blocks. Comitted as obvious. Honza * predict.c (maybe_hot_frequency_p): Do not use cfun. Index: predict.c

Re-enable profile update in tail merging

Hi, this patch re-implements profile update in tail merge which I disabled while converting CFG to profile counts. This is quite easy to do because one only scales probabilities relative to cfg counts. I have stopped on the logic that recomputes counts from edges because it no longer makes much

[Patch, Fortran] PR 82932: [8 Regression] [OOP] ICE in update_compcall_arglist, at fortran/resolve.c:5837

Hi all, here is a close-to-trivial patch for an ICE-on-invalid problem. It regtests cleanly. Ok for trunk? (If there are no objections, I'd like to commit this in the coming days.) Cheers, Janus 2017-11-11 Janus Weil PR fortran/82932 * resolve.c

Re: [PATCH PR82726/PR70754][2/2]New fix by finding correct root reference in combined chains

On Fri, Nov 10, 2017 at 02:14:25PM +, Bin.Cheng wrote: > Hmm, the patch... + /* Setup UID for all statements in dominance order. */ + basic_block *bbs = get_loop_body (loop); + for (i = 0; i < loop->num_nodes; i++) +{ + unsigned uid = 0; + basic_block bb = bbs[i]; + +