[Ping][PATCH v2] Fix Incorrect ASan global variables alignment on arm (PR sanitizer/81697)

2017-11-12 Thread Maxim Ostapenko
Hi, I would like to ping the following patch: https://gcc.gnu.org/ml/gcc-patches/2017-10/msg02288.html Thanks, -Maxim gcc/ChangeLog: 2017-11-13 Maxim Ostapenko PR sanitizer/81697 * asan.c (asan_protect_global): Add new ignore_decl_rtl_set_p parameter. Return

[PATCH] detect nonstring arguments to string functions (PR 82945)

2017-11-12 Thread Martin Sebor
The recently introduced -Wstringop-truncation warning relies on the new nonstring attribute to allow the historical use case of calling strncpy to completely fill the destination with a copy of a string without adding a terminating nul. Glibc is currently considering making use of the attribute

Re: [PATCH v3 1/14] D: The front-end (DMD) language implementation and license.

2017-11-12 Thread Andrei Alexandrescu
On 11/06/2017 01:46 PM, Iain Buclaw wrote: On 25 October 2017 at 03:06, Jeff Law wrote: On 10/18/2017 01:33 AM, Iain Buclaw wrote: On 6 October 2017 at 14:51, Ian Lance Taylor wrote: On Fri, Oct 6, 2017 at 1:34 AM, Iain Buclaw

Re: [riscv] Wrap ASM_OUTPUT_LABELREF in do {} while (0)

2017-11-12 Thread Andrew Waterman
Thanks, Tom. On Sun, Nov 12, 2017 at 8:44 AM, Tom de Vries wrote: > Hi, > > this patch wraps riscv.h's ASM_OUTPUT_LABELREF in "do {} while (0)". > > Build for riscv64. > > Committed as obvious. > > Thanks, > - Tom >

Re: [PATCH] Handle different bit_not_p in store merging (PR tree-optimization/78821)

2017-11-12 Thread Jakub Jelinek
On Fri, Nov 10, 2017 at 04:51:19PM +, Kyrill Tkachov wrote: > Hi Jakub, > > On 10/11/17 13:59, Jakub Jelinek wrote: > > This is something Uros requested in the PR, at least with BIT_NOT_EXPRs > > it is easy. Previous store merging changes required that bit_not_p > > is equal on all stores in

[PATCH] Fix store-merging of cst followed by load (PR tree-optimization/82954)

2017-11-12 Thread Jakub Jelinek
Hi! The conditions split groups if some operand is loaded in first stmt and the second load is incompatible with it, or if it is loaded and second stmt has constant in there instead of load. But as this testcase shows, I didn't handle properly the case when some operand is a constant first and in

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

2017-11-12 Thread Marc Glisse
On Sun, 12 Nov 2017, Martin Sebor wrote: On 11/11/2017 03:03 PM, Marc Glisse wrote: 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

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

2017-11-12 Thread Martin Sebor
On 11/11/2017 03:03 PM, Marc Glisse wrote: 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

[PATCH] gdbinit: break on gfc_internal_error

2017-11-12 Thread Bernhard Reutner-Fischer
Hi! Ok for trunk? gcc/ChangeLog: 2017-11-12 Bernhard Reutner-Fischer * gdbinit.in: Break on gfc_internal_error. Signed-off-by: Bernhard Reutner-Fischer --- gcc/gdbinit.in | 1 + 1 file changed, 1 insertion(+) diff --git

Re: [patch][x86] -march=icelake

2017-11-12 Thread Sandra Loosemore
On 11/11/2017 05:04 PM, Koval, Julia wrote: Hi, this patch adds new option -march=icelake. [snip] diff --git a/gcc/doc/invoke.texi b/gcc/doc/invoke.texi index bc6e86f..891c283 100644 --- a/gcc/doc/invoke.texi +++ b/gcc/doc/invoke.texi @@ -25331,6 +25331,14 @@ RDRND, FMA, BMI, BMI2, F16C,

[riscv] Wrap ASM_OUTPUT_LABELREF in do {} while (0)

2017-11-12 Thread Tom de Vries
Hi, this patch wraps riscv.h's ASM_OUTPUT_LABELREF in "do {} while (0)". Build for riscv64. Committed as obvious. Thanks, - Tom [riscv] Wrap ASM_OUTPUT_LABELREF in do {} while (0) 2017-11-12 Tom de Vries * config/riscv/riscv.h (ASM_OUTPUT_LABELREF): Wrap in do {}

[committed] Remove semicolon after ASM_OUTPUT_ASCII

2017-11-12 Thread Tom de Vries
Hi, this patch removes a semicolon after the ASM_OUTPUT_ASCII macro body in elfos.h. Build for x86_64. Committed as obvious. Thanks, - Tom Remove semicolon after ASM_OUTPUT_ASCII 2017-11-12 Tom de Vries * config/elfos.h (ASM_OUTPUT_ASCII): Remove semicolon after

[cr16, powerpcspe, rs6000] Remove semicolon after ASM_OUTPUT_LABELREF macro body

2017-11-12 Thread Tom de Vries
Hi, this removes a semicolon after the ASM_OUTPUT_LABELREF macro body. Committed as obvious. Thanks, - Tom [cr16, powerpcspe, rs6000] Remove semicolon after ASM_OUTPUT_LABELREF macro body 2017-11-12 Tom de Vries * config/cr16/cr16.h (ASM_OUTPUT_LABELREF): Remove

Re: [patch][x86] -march=icelake

2017-11-12 Thread Uros Bizjak
On Sun, Nov 12, 2017 at 1:04 AM, Koval, Julia wrote: > 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

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

2017-11-12 Thread Uros Bizjak
On Sat, Nov 11, 2017 at 10:10 PM, Koval, Julia wrote: > 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? It looks like an oversight to me. If there are no "skylake-avx512" model, then the

Re: [PATCH] Fixes for PR68356, PR81210, and PR81693

2017-11-12 Thread H.J. Lu
On Sun, Nov 12, 2017 at 6:22 AM, Dominique d'Humières wrote: > The following patch fixes PR68356, PR81210, and PR81693 on darwin. > > --- ../_clean/gcc/testsuite/gcc.dg/torture/pr68264.c2016-01-28 > 00:30:03.0 +0100 > +++ gcc/testsuite/gcc.dg/torture/pr68264.c

[PATCH] Fixes for PR68356, PR81210, and PR81693

2017-11-12 Thread Dominique d'Humières
The following patch fixes PR68356, PR81210, and PR81693 on darwin. --- ../_clean/gcc/testsuite/gcc.dg/torture/pr68264.c2016-01-28 00:30:03.0 +0100 +++ gcc/testsuite/gcc.dg/torture/pr68264.c 2017-11-11 17:16:58.0 +0100 @@ -1,4 +1,5 @@ /* { dg-do run } */ +/* {

[PATCH] Fix pr81706 tests on darwin

2017-11-12 Thread Dominique d'Humières
The following patch fixes pr81706 tests on darwin --- ../_clean/gcc/testsuite/gcc.target/i386/pr81706.c 2017-10-26 07:16:18.0 +0200 +++ gcc/testsuite/gcc.target/i386/pr81706.c 2017-11-11 16:02:36.0 +0100 @@ -1,8 +1,8 @@ /* PR libstdc++/81706 */ /* { dg-do compile } */

[RFC gfortran] PR53478 - gfortran segfaults when module name clashes with C binding name of procedure

2017-11-12 Thread Dominique d'Humières
This patch implement the requirement > ... Furthermore, a binding label shall not be > the same as the global identifier of any other global entity, ignoring > differences in case." While looking at the code, I noticed that several %s should be %qs. This is fixed as well by the patch along with

[patch][x86,avx] Enable AVX512BITALG

2017-11-12 Thread Koval, Julia
Hi, this patch enables AVX512BITALG and AVX512VPOPCNTDQ instructions from https://software.intel.com/sites/default/files/managed/c5/15/architecture-instruction-set-extensions-programming-reference.pdf. Ok for trunk? Thanks, Julia Julia Koval Sebastian Peryt

Re: Drop frequencies from cgraph edges

2017-11-12 Thread Andreas Schwab
This causes bootstrap comparison failures on ia64. For example: $ diff -u <(nm stage{2,3}-gcc/gcov.o) --- /dev/fd/63 2017-11-12 12:12:14.174568108 +0100 +++ /dev/fd/62 2017-11-12 12:12:14.174568108 +0100 @@ -33,9 +33,9 @@ 0050 s _ZL15flag_long_names 0044 s