Re: Reorganise machmode.h headers

2017-06-11 Thread Richard Sandiford
Ping Richard Sandiford writes: > Jeff Law writes: >> On 11/16/2016 09:32 AM, Richard Sandiford wrote: >>> Later patches will make machmode.h rely on wide-int.h and the >>> new poly-int.h, so it needs to appear later in the coretypes.h >>> include list. >>> >>> Previously machmode.h included insn

Re: Fix pessimistic DImode handling in combine.c:make_field_assignment

2017-06-11 Thread Richard Sandiford
Ping Richard Sandiford writes: > Richard Sandiford writes: >> The make_field_assignment code: >> >> src = force_to_mode (src, mode, >> GET_MODE_PRECISION (mode) >= HOST_BITS_PER_WIDE_INT >> ? HOST_WIDE_INT_M1U >> : (HOST_WIDE_INT_

Re: [2/2] PR 80769: Incorrect strlen optimisation

2017-06-11 Thread Richard Sandiford
Ping*2 Richard Sandiford writes: > In this testcase, we (correctly) record after: > > strcpy (p1, "abcde"); > char *p2 = strchr (p1, '\0'); > strcpy (p2, q); > > that the length of p1 and p2 can be calculated by converting the > second strcpy to: > > tmp = stpcpy (p2, q) > > and then doin

Re: [patch,avr] Add support for devices with flash accessible by LD.

2017-06-11 Thread Pitchumani Sivanupandi
On Friday 09 June 2017 03:59 PM, Georg-Johann Lay wrote: Hi, This patch adds support for devices that can access flash memory by LD* instructions, hence there is no need to put .rodata in RAM. The default linker script for the new multilib versions already supports this feature, it's similar to

libbacktrace patch committed: Fix race on parallel initialization

2017-06-11 Thread Ian Lance Taylor
The code in libbacktrace had a race when doing parallel initialization: if two threads start to initialize at the same time, and one completes first, the other, while running in backtrace_initialize, may see that the structure is initialized and thus not change *fileline_fn. The caller will then s

Re: Patch RFC: disable block partitioning with split stack

2017-06-11 Thread Ian Lance Taylor
On Sun, Jun 11, 2017 at 2:38 AM, Jan Hubicka wrote: >> > >> > Thanks for explanation. Perhaps we could have this documented, because >> > otherwise people will think the option is simply broken. I guess even >> > better >> > we could have configure autodetection for the broken linker. >> >> Comm

Re: [PATCH] Fix new split-1.c testcase

2017-06-11 Thread Ian Lance Taylor
On Sun, Jun 11, 2017 at 4:40 AM, Segher Boessenkool wrote: > > The new split-1.c testcase fails on targets that do not support split > stack (like 32-bit PowerPC Linux). This patch fixes it by only running > the testcase if split stack is supported. It also adds the reorder > flag to the options

[PATCH 6/6] New warnings -Wstring-plus-{char, int} (PR c++/62181)

2017-06-11 Thread Xi Ruoyao
This patch adds document of -Wstring-plus-int and -Wstring-plus-char. gcc/ChangeLog: 2017-06-12  Xi Ruoyao   * doc/invoke.texi (Warning Options): Document -Wstring-plus-int and -Wstring-plus-char. ---  gcc/doc/invoke.texi | 22 +-  1 file changed, 21 insertions

[PATCH 5/6] New warnings -Wstring-plus-{char, int} (PR c++/62181)

2017-06-11 Thread Xi Ruoyao
This patch adds tests for -Wstring-plus-char. gcc/ChangeLog: 2017-06-12  Xi Ruoyao   * testsuite/c-c++-common/Wstring-plus-char.c: New test. * testsuite/g++.dg/Wstring-plus-char-1.C: Ditto. * testsuite/g++.dg/Wstring-plus-char-2.C: Ditto. * testsuite/g++.dg/Wstrin

[PATCH 4/6] New warnings -Wstring-plus-{char, int} (PR c++/62181)

2017-06-11 Thread Xi Ruoyao
This patch adds tests for -Wstring-plus-int. gcc/ChangeLog: 2017-06-12  Xi Ruoyao   * testsuite/c-c++-common/Wstring-plus-int.c: New test. * testsuite/g++.dg/Wstring-plus-int-1.C: Ditto. * testsuite/g++.dg/Wstring-plus-int-2.C: Ditto. ---  gcc/testsuite/c-c++-common/Wstri

[PATCH 3/6] New warnings -Wstring-plus-{char, int} (PR c++/62181)

2017-06-11 Thread Xi Ruoyao
This patch adds warning option -Wstring-plus-char for C/C++. gcc/ChangeLog: 2017-06-12  Xi Ruoyao   * c-family/c-common.h (warn_if_string_plus_char): New prototype. * c-family/c-warn.c (warn_if_string_plus_char): New function. * c-family/c.opt: New option -Wstring-plus-ch

[PATCH 2/6] New warnings -Wstring-plus-{char, int} (PR c++/62181)

2017-06-11 Thread Xi Ruoyao
This patch adds warning option -Wstring-plus-int for C/C++. gcc/ChangeLog: 2017-06-12  Xi Ruoyao   * c-family/c.opt: New option -Wstring-plus-int. * c-family/c-common.c (pointer_int_sum): Checking for -Wstring-plus-int. ---  gcc/c-family/c-common.c | 25 ++

[PATCH 1/6] New warnings -Wstring-plus-{char, int} (PR c++/62181)

2017-06-11 Thread Xi Ruoyao
This patch moves prototype of char_type_p into c-common.h, so we can use it in c-family/*. gcc/ChangeLog: 2017-06-12  Xi Ruoyao   * c-family/c-common.h (char_type_p): New prototype. * c/c-typeck.c (char_type_p): Make the function extern. * cp/cp-tree.h (char_type_p): Remo

[PATCH 0/6] New warnings -Wstring-plus-{char, int} (PR c++/62181)

2017-06-11 Thread Xi Ruoyao
Hi, I've implemented -Wstring-plus-int and -Wstring-plus-char (like their counterpart in Clang) for GCC. This series of patch has been bootstrapped and regtested.  OK for trunk? Currently these options are not enabled by default like Clang does. Maybe we could make them enabled by default or by

Re: [PATCH PR78005]Fix miscompare issue by computing correct guard condition for vectorized loop

2017-06-11 Thread Bin.Cheng
On Sat, Jun 10, 2017 at 10:40 AM, Richard Sandiford wrote: > Sorry to return this old patch, but: > > Bin Cheng writes: >> -/* Calculate the number of iterations under which scalar loop will be >> - preferred than vectorized loop. NITERS_PROLOG is the number of >> - iterations of prolog loop

[nvptx, committed, PR79939] Disable constant pool for nvptx

2017-06-11 Thread Tom de Vries
Hi, this patch fixes PR79939, a cc1 hang while trying to emit a constant in the constant pool. Fixed by disabling the constant pool for nvptx. Tested on target nvptx. Committed as obvious. Thanks, - Tom Disable constant pool for nvptx 2017-06-11 Tom de Vries PR target/79939 * config/

[PATCH try 2 resend] [i386] Remove warnings for ignoring -mcall-ms2sysv-xlogues.

2017-06-11 Thread Daniel Santos
I appear to have forgotten to cc gcc-patches, sorry about that. There are currently three cases where we issue a warning when disabling -mcall-ms2sysv-xlogues for a function, but I never added a proper warning, so there's no mechanism for disabling it. This is something that I meant to address so

Re: [PATCH doc] update documentation of x86 -mcx16 option

2017-06-11 Thread Gerald Pfeifer
On Sun, 4 Jun 2017, Sandra Loosemore wrote: > This is good, thanks. I think it's fine for GCC 7 branch as well (I guess > it's not a regression fix, but it seems unlikely to break anything). Plus the regression-only rule does not apply for docs. ;-) Gerald

Re: MAINTAINERS update

2017-06-11 Thread Gerald Pfeifer
On Tue, 30 May 2017, Bernd Schmidt wrote: > On 05/30/2017 09:05 AM, Richard Biener wrote: >> This leaves the nvptx and c6x ports without a maintainer. Do >> you have any recommendations for a successor here? > Not really. It would be a shame to lose the C6X port though. If I'm > CC'd on any bug

Re: [PATCH,Testsuite] Check split_stack is ok for target in tree-prof/split-1.c

2017-06-11 Thread Rainer Orth
Hi Paul, > tree-prof/split-1.c use -fsplit-stack in dg-options but not check is > ok for target. > This patch add "dg-require-effective-target split_stack" for it. > > Ok for commit ? ok with the ChangeLog nit below fixed. Thanks. Rainer > ChangeLog > 2017-06-11 Chenghua Xu

[PATCH] Fix new split-1.c testcase

2017-06-11 Thread Segher Boessenkool
The new split-1.c testcase fails on targets that do not support split stack (like 32-bit PowerPC Linux). This patch fixes it by only running the testcase if split stack is supported. It also adds the reorder flag to the options, so that the test actually tests what it says it tests. Is this okay

Re: Patch RFC: disable block partitioning with split stack

2017-06-11 Thread Jan Hubicka
> > > > Thanks for explanation. Perhaps we could have this documented, because > > otherwise people will think the option is simply broken. I guess even better > > we could have configure autodetection for the broken linker. > > Committed to mainline with docs as follows. Thanks, the patch howev

[PATCH,Testsuite] Check split_stack is ok for target in tree-prof/split-1.c

2017-06-11 Thread Paul Hua
Hi: tree-prof/split-1.c use -fsplit-stack in dg-options but not check is ok for target. This patch add "dg-require-effective-target split_stack" for it. Ok for commit ? Paul. ChangeLog 2017-06-11 Chenghua Xu * gcc.dg/tree-prof/split-1.c: Require split_stack support. diff --git