[PATCH] For broken exception handling in GDB on AIX platform

2017-02-28 Thread Nitish Kumar Mishra
Hi, The patch is for the broken exception handling in GDB on AIX platform. When linked statically with libstdc++ and libgcc on AIX platform, GDB is facing broken exception handling issues. Following is the error output when GDB is linked statically with mentioned libraries: (GDB-7.12.1, built with

Re: PR79715: Special case strcpy/strncpy for dse

2017-02-28 Thread Richard Biener
On Tue, 28 Feb 2017, Jeff Law wrote: > On 02/28/2017 05:59 AM, Prathamesh Kulkarni wrote: > > On 28 February 2017 at 15:40, Jakub Jelinek wrote: > > > On Tue, Feb 28, 2017 at 03:33:11PM +0530, Prathamesh Kulkarni wrote: > > > > Hi, > > > > The attached patch adds special-casing

Re: [PATCH] Fix vector lowering of VEC_COND_EXPR with VECTOR_BOOLEAN_TYPE_P with scalar mode (PR tree-optimization/79734)

2017-02-28 Thread Richard Biener
On Tue, 28 Feb 2017, Jakub Jelinek wrote: > Hi! > > This patch fixes ICE during lowering of VEC_COND_EXPR which has the > AVX512-ish [QHSD]Imode VECTOR_BOOLEAN_TYPE_P type (where the TYPE_SIZE > of the comp_inner_type and inner_type are different). > In addition, it attempts to expand it into

Re: [PATCH docs] remove Java from GCC 7 release criteria

2017-02-28 Thread Richard Biener
On March 1, 2017 3:34:46 AM GMT+01:00, Martin Sebor wrote: >On 02/28/2017 01:41 PM, Richard Biener wrote: >> On February 28, 2017 7:00:39 PM GMT+01:00, Jeff Law >wrote: >>> On 02/28/2017 10:54 AM, Martin Sebor wrote: The GCC 7 release criteria page

[PATCH], PR target/79434, fix PowerPC recursive calls that can replaced at runtime

2017-02-28 Thread Michael Meissner
This patch fixes PR target/79439, which is a recursive call when the 64-bit code is compiled with -fpic doesn't have the NOP after the call. It is possible for the function to be overriden at link time. In such a case, the call should call the module that is overriding the call, rather than

Re: ARC options documentation questions

2017-02-28 Thread Sandra Loosemore
On 02/24/2017 12:20 PM, Claudiu Zissulescu wrote: Hi, Indeed, we are not up to speed regarding updating and cleaning the documentation. On 12/02/2017 05:18, Sandra Loosemore wrote: I noticed a bunch of copy-editing issues in the "ARC Options" section of invoke.texi. I'm willing to take a

Re: [PATCH docs] remove Java from GCC 7 release criteria

2017-02-28 Thread Martin Sebor
On 02/28/2017 01:41 PM, Richard Biener wrote: On February 28, 2017 7:00:39 PM GMT+01:00, Jeff Law wrote: On 02/28/2017 10:54 AM, Martin Sebor wrote: The GCC 7 release criteria page mentions Java even though the front end has been removed. The attached patch removes Java from

Re: [PATCH,rs6000] PR79395: Fix compile error with -mcpu=power9 and -mno-vsx and __builtin_vec_cmpne_p

2017-02-28 Thread Segher Boessenkool
Hi Kelvin, On Tue, Feb 28, 2017 at 03:46:20PM -0700, Kelvin Nilsen wrote: > PR 79395 reports a problem that arises when the preprocessor believes > that the target supports Power9 but the gcc compiler believes that > Power9 is not supported. > > This patch addresses this inconsistency by

Re: [C++ PATCH] Fix -Wunused-but-set-parameter in ctors of some abstract classes (PR c++/79746)

2017-02-28 Thread Nathan Sidwell
On 02/28/2017 02:41 PM, Jason Merrill wrote: On Tue, Feb 28, 2017 at 12:48 PM, Jakub Jelinek wrote: The DR1659/DR1611 changes result in construct_virtual_base not being called, but unfortunately the call generated in there was the spot that caused mark_exp_read on the

Re: [C++ PATCH] Fix -Wunused-but-set-parameter in ctors of some abstract classes (PR c++/79746)

2017-02-28 Thread Jason Merrill
On Tue, Feb 28, 2017 at 12:48 PM, Jakub Jelinek wrote: > The DR1659/DR1611 changes result in construct_virtual_base not being called, > but unfortunately the call generated in there was the spot that caused > mark_exp_read on the arguments passed to the vbase construction

Re: [PATCH][PR target/79752] fix rs6000 power9 peephole2 for udiv/umod

2017-02-28 Thread Segher Boessenkool
On Tue, Feb 28, 2017 at 04:02:00PM -0600, Aaron Sawdey wrote: > This showed up in power9 code for __divkf3 software float support and > caused a divd to be emitted where we needed a divdu. OK for trunk if > bootstrap/regtest passes? Yes, certainly. Thanks! Segher > ---

C++ PATCH for C++17 class template argument deduction issues

2017-02-28 Thread Jason Merrill
This patch implements some proposed resolutions to open issues with C++17 class template argument deduction. Tested x86_64-pc-linux-gnu, applying to trunk. commit 5486547c53bf6e83bcdd822b2f9cdce171d115b0 Author: Jason Merrill Date: Tue Feb 28 07:45:04 2017 -1000

Re: C++ PATCH to fix wrong-code with pointer-to-data-members (PR c++/79687)

2017-02-28 Thread Jason Merrill
On Tue, Feb 28, 2017 at 10:10 AM, Marek Polacek wrote: > On Fri, Feb 24, 2017 at 11:11:05AM -0800, Jason Merrill wrote: >> On Fri, Feb 24, 2017 at 8:22 AM, Marek Polacek wrote: >> > I had an interesting time tracking down some of the problems with this >>

[C++ PATCH] Fix -Wunused-but-set-parameter in ctors of some abstract classes (PR c++/79746)

2017-02-28 Thread Jakub Jelinek
Hi! The DR1659/DR1611 changes result in construct_virtual_base not being called, but unfortunately the call generated in there was the spot that caused mark_exp_read on the arguments passed to the vbase construction (TREE_USED is set on these earlier already during parsing them). That results in

[PATCH,rs6000] PR79395: Fix compile error with -mcpu=power9 and -mno-vsx and __builtin_vec_cmpne_p

2017-02-28 Thread Kelvin Nilsen
PR 79395 reports a problem that arises when the preprocessor believes that the target supports Power9 but the gcc compiler believes that Power9 is not supported. This patch addresses this inconsistency by introducing a new preprocessor macro named __POWER9_VECTOR__ which is automatically defined

[PATCH] Fix vector lowering of VEC_COND_EXPR with VECTOR_BOOLEAN_TYPE_P with scalar mode (PR tree-optimization/79734)

2017-02-28 Thread Jakub Jelinek
Hi! This patch fixes ICE during lowering of VEC_COND_EXPR which has the AVX512-ish [QHSD]Imode VECTOR_BOOLEAN_TYPE_P type (where the TYPE_SIZE of the comp_inner_type and inner_type are different). In addition, it attempts to expand it into efficient code when possible (by performing the

Re: libgo patch committed: Fix quoting in mksigtab.sh

2017-02-28 Thread Ian Lance Taylor
On Tue, Feb 28, 2017 at 12:08 PM, Rainer Orth wrote: > Hi Ian, > >> I goofed on the shell quoting in mksigtab.sh, causing various >> system-specific signals to be entered incorrectly in the runtime >> signal table. This patch fixes the problem. The test for this

[Ada] PR target/79749

2017-02-28 Thread Eric Botcazou
/79749 * config/sparc/sparc.c (sparc_frame_pointer_required): Add missing condition on optimize for the leaf function test. 2017-02-28 Eric Botcazou <ebotca...@adacore.com> * gcc.target/sparc/20170228-1.c: New test. -- Eric Botcazou/* PR target/79749 */ /* Reported by Ra

[PATCH][PR target/79752] fix rs6000 power9 peephole2 for udiv/umod

2017-02-28 Thread Aaron Sawdey
This showed up in power9 code for __divkf3 software float support and caused a divd to be emitted where we needed a divdu. OK for trunk if bootstrap/regtest passes? Index: ../trunk/gcc/config/rs6000/rs6000.md === ---

Re: PR79697: Delete calls to strdup, strndup, realloc if there is no lhs

2017-02-28 Thread Peter Bergner
On 2/28/17 12:49 PM, Peter Bergner wrote: > On 2/25/17 2:40 AM, Prathamesh Kulkarni wrote: >> The attached patch deletes calls to strdup, strndup if it's >> return-value is unused, >> and same for realloc if the first arg is NULL. > > Why limit ourselves to strdup and strndup? Can't we do the

Re: [PATCH docs] remove Java from GCC 7 release criteria

2017-02-28 Thread Richard Biener
On February 28, 2017 7:00:39 PM GMT+01:00, Jeff Law wrote: >On 02/28/2017 10:54 AM, Martin Sebor wrote: >> The GCC 7 release criteria page mentions Java even though >> the front end has been removed. The attached patch removes Java >> from the criteria page. While reviewing the

Re: C++ PATCH to fix wrong-code with pointer-to-data-members (PR c++/79687)

2017-02-28 Thread Marek Polacek
On Fri, Feb 24, 2017 at 11:11:05AM -0800, Jason Merrill wrote: > On Fri, Feb 24, 2017 at 8:22 AM, Marek Polacek wrote: > > I had an interesting time tracking down some of the problems with this code. > > Hopefully I've sussed out now how this stuff works. > > > > We've got > >

Re: libgo patch committed: Fix quoting in mksigtab.sh

2017-02-28 Thread Rainer Orth
Hi Ian, > I goofed on the shell quoting in mksigtab.sh, causing various > system-specific signals to be entered incorrectly in the runtime > signal table. This patch fixes the problem. The test for this is > misc/cgo/testcarchive in the master repo, which > we don't yet run for gccgo.

Fwd: [Bug fortran/79739] [7 Regression] ICE with some interesting code

2017-02-28 Thread Paul Richard Thomas
Patch committed as obvious/trivial. I didn't think that it warranted a testcase since there is no way that this one will come back. Thanks for the report. Paul -- Forwarded message -- From: pault at gcc dot gnu.org Date: 28 February 2017 at 19:32

Merge from trunk to gccgo branch

2017-02-28 Thread Ian Lance Taylor
I've merged trunk revision 245784 to the gccgo branch. Ian

Re: PR79697: Delete calls to strdup, strndup, realloc if there is no lhs

2017-02-28 Thread Peter Bergner
On 2/25/17 2:40 AM, Prathamesh Kulkarni wrote: > The attached patch deletes calls to strdup, strndup if it's > return-value is unused, > and same for realloc if the first arg is NULL. Why limit ourselves to strdup and strndup? Can't we do the same for all functions that are marked as const/pure

Re: [PATCH docs] remove Java from GCC 7 release criteria

2017-02-28 Thread Jeff Law
On 02/28/2017 11:09 AM, Martin Sebor wrote: On 02/28/2017 11:00 AM, Jeff Law wrote: On 02/28/2017 10:54 AM, Martin Sebor wrote: The GCC 7 release criteria page mentions Java even though the front end has been removed. The attached patch removes Java from the criteria page. While reviewing

Re: [PATCH 2/3] Introduce = for couple of options.

2017-02-28 Thread Joseph Myers
On Tue, 28 Feb 2017, Martin Liška wrote: > Hello. > > This is second patch, where I add '=' for some of options. OK. -- Joseph S. Myers jos...@codesourcery.com

Re: [PATCH] Add tabulars for options with enum values.

2017-02-28 Thread Joseph Myers
On Tue, 28 Feb 2017, Martin Liška wrote: > Ready to be installed after regression tests? OK. -- Joseph S. Myers jos...@codesourcery.com

Re: [PATCH docs] remove Java from GCC 7 release criteria

2017-02-28 Thread Martin Sebor
On 02/28/2017 11:00 AM, Jeff Law wrote: On 02/28/2017 10:54 AM, Martin Sebor wrote: The GCC 7 release criteria page mentions Java even though the front end has been removed. The attached patch removes Java from the criteria page. While reviewing the rest of the text I noticed a few minor

Re: [PATCH docs] remove Java from GCC 7 release criteria

2017-02-28 Thread Jeff Law
On 02/28/2017 10:54 AM, Martin Sebor wrote: The GCC 7 release criteria page mentions Java even though the front end has been removed. The attached patch removes Java from the criteria page. While reviewing the rest of the text I noticed a few minor typos that I corrected in the patch as well.

Re: PR79715: Special case strcpy/strncpy for dse

2017-02-28 Thread Jeff Law
On 02/28/2017 05:59 AM, Prathamesh Kulkarni wrote: On 28 February 2017 at 15:40, Jakub Jelinek wrote: On Tue, Feb 28, 2017 at 03:33:11PM +0530, Prathamesh Kulkarni wrote: Hi, The attached patch adds special-casing for strcpy/strncpy to dse pass. Bootstrapped+tested on

[PATCH docs] remove Java from GCC 7 release criteria

2017-02-28 Thread Martin Sebor
The GCC 7 release criteria page mentions Java even though the front end has been removed. The attached patch removes Java from the criteria page. While reviewing the rest of the text I noticed a few minor typos that I corrected in the patch as well. Btw., as an aside, I read the page to see if

OpenACC 2.5 Profiling Interface (incomplete)

2017-02-28 Thread Thomas Schwinge
Hi! The 2.5 versions of the OpenACC standard added a new chapter "Profiling Interface". In r245784, I committed incomplete support to gomp-4_0-branch. I plan to continue working on this, but wanted to synchronize at this point. commit b22a85fe7f3daeb48460e7aa28606d0cdb799f69 Author: tschwinge

libgomp, OpenACC: Fix locking of cached_base_dev (guarded by acc_device_lock)

2017-02-28 Thread Thomas Schwinge
Hi! We'd noticed before some locking issues in libgomp; in r245783 I just committed the following to gomp-4_0-branch: commit 921620f947485002e56f736d4ca18bd2ae33e05b Author: tschwinge Date: Tue Feb 28 17:35:49 2017 + libgomp, OpenACC:

Re: [RFC PATCH 2/3] Use call_summary in ipa-prop and ipa-cp

2017-02-28 Thread Martin Jambor
Hi, On Tue, Feb 28, 2017 at 11:50:01AM +0100, jh wrote: > Dne 2017-02-27 17:35, Martin Jambor napsal: > > Hello, > > > > this is patch is afairly straightforward conversion from use of a > > vector indexed by edge->uid to use of the new call_summary from the > > previous patch. > > > > The

Re: [PATCH] Fix gimple-ssa-store-merging.c on little-endian (PR tree-optimization/79737)

2017-02-28 Thread Kyrill Tkachov
On 28/02/17 16:29, Richard Biener wrote: On February 28, 2017 5:00:41 PM GMT+01:00, Jakub Jelinek wrote: Hi! The following testcases are miscompiled on little endian targets. The bug occurs if we are merging bitfield stores, there is a signed bitfield with bitlen multiple

Re: [PATCH] Workaround premature bitfield folding (PR c++/79681)

2017-02-28 Thread Jakub Jelinek
On Tue, Feb 28, 2017 at 04:43:19PM +0100, Richard Biener wrote: > > This patch just attempts to reuse as much as possible from orig_inner and > > only use adjusted BIT_FIELD_REF on that when in GENERIC (first I've tried to > > use instead COMPONENT_REF with DECL_BIT_FIELD_REPRESENTATIVE, but that

Re: [PATCH] Fix gimple-ssa-store-merging.c on little-endian (PR tree-optimization/79737)

2017-02-28 Thread Richard Biener
On February 28, 2017 5:00:41 PM GMT+01:00, Jakub Jelinek wrote: >Hi! > >The following testcases are miscompiled on little endian targets. >The bug occurs if we are merging bitfield stores, there is a signed >bitfield >with bitlen multiple of BITS_PER_UNIT but not equal to the

Re: [PATCH] Another x86 output_operand_lossage fix (PR target/79729)

2017-02-28 Thread Uros Bizjak
On Tue, Feb 28, 2017 at 5:04 PM, Jakub Jelinek wrote: > Hi! > > Apparently when fixing these issues earlier I've missed one spot with > gcc_unreachable () dependent on the value that could be provided by user > in bogus inline asm. > > Bootstrapped/regtested on x86_64-linux and

[PATCH] Another x86 output_operand_lossage fix (PR target/79729)

2017-02-28 Thread Jakub Jelinek
Hi! Apparently when fixing these issues earlier I've missed one spot with gcc_unreachable () dependent on the value that could be provided by user in bogus inline asm. Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk? 2017-02-28 Jakub Jelinek PR

[PATCH] [ARC] Handle complex PIC move patterns.

2017-02-28 Thread Claudiu Zissulescu
Hi, fwprop step is placing in the REG_EQUIV notes constant pic unspecs expressions. Then, loop may use these notes for optimizations rezulting in complex patterns that are not supported by the current implementation. The patch adds handling of complex PIC addresses having MINUS or UNARY

[PATCH] Fix gimple-ssa-store-merging.c on little-endian (PR tree-optimization/79737)

2017-02-28 Thread Jakub Jelinek
Hi! The following testcases are miscompiled on little endian targets. The bug occurs if we are merging bitfield stores, there is a signed bitfield with bitlen multiple of BITS_PER_UNIT but not equal to the bitsize of the corresponding mode, the bitfield doesn't start on multiple of BITS_PER_UNIT,

[PATCH 0/2] [ARC] Code size mods.

2017-02-28 Thread Claudiu Zissulescu
Hi, This series of two patches adds new ARCv2 code-density instructions to our backend (patch 1). And improves the size figures by using section anchors and different register allocation order (patch 2). OK to apply? Claudiu Claudiu Zissulescu (2): [ARC] Add code density instructions. [ARC]

[PATCH 1/2] [ARC] Add code density instructions.

2017-02-28 Thread Claudiu Zissulescu
gcc/ 2016-09-16 Claudiu Zissulescu * config/arc/arc.c (arc_output_addsi): Emit code density adds. * config/arc/arc.md (cpu_facility): Add cd variant. (*movqi_insn): Add code density variant. (*movhi_insn): Likewise. (*movqi_insn):

[PATCH 2/2] [ARC] Code size modifications.

2017-02-28 Thread Claudiu Zissulescu
gcc/ 2016-09-20 Claudiu Zissulescu * config/arc/arc.c (arc_init): Use multiplier whenever we have it. (arc_conditional_register_usage): Use a different allocation order when optimizing for size. * common/config/arc/arc-common.c

Re: [PATCH] Do not allow to run GIMPLE or RTL test-cases w/ LTO (PR lto/79625).

2017-02-28 Thread Richard Biener
On Tue, Feb 28, 2017 at 4:39 PM, Martin Liška wrote: > On 02/28/2017 04:36 PM, Richard Biener wrote: >> On Tue, Feb 28, 2017 at 4:15 PM, Martin Liška wrote: >>> On 02/28/2017 03:45 PM, Richard Biener wrote: On Tue, Feb 28, 2017 at 2:51 PM, Martin Liška

Re: [PATCH] Workaround premature bitfield folding (PR c++/79681)

2017-02-28 Thread Richard Biener
On Tue, 28 Feb 2017, Jakub Jelinek wrote: > Hi! > > make_bit_field_ref and its callers are invoked not just during GIMPLE (where > at least fold_truth_andor* isn't really effective; but from what I've tested > it doesn't do a good job in many cases at GENERIC either, because we > fold x.btfld ==

Re: [PATCH] Do not allow to run GIMPLE or RTL test-cases w/ LTO (PR lto/79625).

2017-02-28 Thread Martin Liška
On 02/28/2017 04:36 PM, Richard Biener wrote: > On Tue, Feb 28, 2017 at 4:15 PM, Martin Liška wrote: >> On 02/28/2017 03:45 PM, Richard Biener wrote: >>> On Tue, Feb 28, 2017 at 2:51 PM, Martin Liška wrote: Hello. As mentioned in the PR, I hope

Re: [RFA PATCH, i386]: Warn for 64-bit values in general-reg asm operands and error out for 8-bit values in invalid GR asm operand

2017-02-28 Thread Jakub Jelinek
On Tue, Feb 28, 2017 at 04:34:20PM +0100, Uros Bizjak wrote: > > Have you tried to build say Linux kernel or firefox or similar large > > codebase with lots of inline asm with that? > > No... > > > What constraint should people use for long long vars in 32-bit code? > > "A" constraint is used a

Re: [PATCH] Do not allow to run GIMPLE or RTL test-cases w/ LTO (PR lto/79625).

2017-02-28 Thread Richard Biener
On Tue, Feb 28, 2017 at 4:15 PM, Martin Liška wrote: > On 02/28/2017 03:45 PM, Richard Biener wrote: >> On Tue, Feb 28, 2017 at 2:51 PM, Martin Liška wrote: >>> Hello. >>> >>> As mentioned in the PR, I hope rejecting -flto option with both GIMPLE and >>> RTL can

[PATCH] Workaround premature bitfield folding (PR c++/79681)

2017-02-28 Thread Jakub Jelinek
Hi! make_bit_field_ref and its callers are invoked not just during GIMPLE (where at least fold_truth_andor* isn't really effective; but from what I've tested it doesn't do a good job in many cases at GENERIC either, because we fold x.btfld == y.btfld too early into ((BIT_FIELD_REF ^

[PATCH] Fix PR79721

2017-02-28 Thread Richard Biener
The following fixes final value replacement introducing undefined overflow. The issue is in how we compute Newtons interpolating formula which really needs to be done in a type that has well-defined overflow behavior. Bootstrapped on x86_64-unknown-linux-gnu, testing in progress. Eventually

Re: [RFA PATCH, i386]: Warn for 64-bit values in general-reg asm operands and error out for 8-bit values in invalid GR asm operand

2017-02-28 Thread Uros Bizjak
On Tue, Feb 28, 2017 at 12:06 PM, Jakub Jelinek wrote: > On Tue, Feb 28, 2017 at 11:41:56AM +0100, Richard Biener wrote: >> > 2017-02-28 Uros Bizjak >> > >> > * config/i386/i386.c (print_reg): Warn for values of 64-bit size >> > in integer register

[PATCH 1/2] [ARC] Fix conditional move contstraint

2017-02-28 Thread Claudiu Zissulescu
Move pattern (movsi_insn) allows predicated instructions to be instructions which can hold all registers. However, the conditional variant doesn't. This patch fixes this problem. 2017-02-28 Claudiu Zissulescu * config/arc/arc.md (movsi_cond_exec): Update

[PATCH 0/2] [ARC] Two small patches to fix faulty instruction patterns.

2017-02-28 Thread Claudiu Zissulescu
Hi, This series of two patches fixes a typo in tst instruction, and a faulty mov conditional instruction. Ok to apply? Claudiu Claudiu Zissulescu (2): [ARC] Fix conditional move contstraint Fix tst_bitfield_tst pattern. gcc/config/arc/arc.md | 4 ++--

[PATCH 2/2] [ARC] Fix tst_bitfield_tst pattern.

2017-02-28 Thread Claudiu Zissulescu
Fixes a typo, adds a test. gcc/ 2017-02-28 Claudiu Zissulescu * config/arc/arc.md (*tst_bitfield_tst): Fix pattern. gcc/testsuite 2017-02-28 Claudiu Zissulescu * gcc.target/arc/bitfield.c: New file. --- gcc/config/arc/arc.md

Re: [PATCH] Fix PR79740

2017-02-28 Thread Richard Biener
On Tue, 28 Feb 2017, Richard Biener wrote: > > Bootstrap / regtest running on x86_64-unknown-linux-gnu. > > Richard. The following is what I committed. Bootstrapped and tested on x86_64-unknown-linux-gnu. Richard. 2017-02-28 Richard Biener PR

Re: [PATCH] Fix PR79731

2017-02-28 Thread Richard Biener
On Tue, 28 Feb 2017, Richard Biener wrote: > > Bootstrap / regtest running on x86_64-unknown-linux-gnu. The following is what I committed. Bootstrapped and tested on x86_64-unknown-linux-gnu. Richard. 2017-02-28 Richard Biener PR middle-end/79731 *

RE: [PATCH,testsuite] Skip gcc.dg/lto/pr60449_0.c for mips*-*-elf* targets.

2017-02-28 Thread Moore, Catherine
> -Original Message- > From: Toma Tabacu [mailto:toma.tab...@imgtec.com] > Sent: Tuesday, February 28, 2017 9:32 AM > To: gcc-patches@gcc.gnu.org > Cc: Matthew Fortune ; Moore, > Catherine > Subject: [PATCH,testsuite] Skip

Re: [PATCH] Do not allow to run GIMPLE or RTL test-cases w/ LTO (PR lto/79625).

2017-02-28 Thread Martin Liška
On 02/28/2017 03:45 PM, Richard Biener wrote: > On Tue, Feb 28, 2017 at 2:51 PM, Martin Liška wrote: >> Hello. >> >> As mentioned in the PR, I hope rejecting -flto option with both GIMPLE and >> RTL can >> work for the issue. >> >> Ready after it finishes regression tests? > >

libgo patch committed: Fix sigfwd to not allocate memory

2017-02-28 Thread Ian Lance Taylor
This patch to libgo fixes sigfwd to not allocate memory. The use of &[1]uintptr{fn} was causing memory allocation, even though it is being compiled for the runtime package. That is a bad idea for this function, which is invoked by a signal handler. Rewrite it to use only constructs that do not

libgo patch committed: Fix quoting in mksigtab.sh

2017-02-28 Thread Ian Lance Taylor
I goofed on the shell quoting in mksigtab.sh, causing various system-specific signals to be entered incorrectly in the runtime signal table. This patch fixes the problem. The test for this is misc/cgo/testcarchive in the master repo, which we don't yet run for gccgo. Bootstrapped and ran Go

Re: [PR 78140] Reuse same IPA bits and VR info

2017-02-28 Thread Richard Biener
On Tue, Feb 28, 2017 at 3:08 PM, Martin Jambor wrote: > Hi, > > On Mon, Feb 27, 2017 at 10:34:38AM +0100, Richard Biener wrote: >> On Wed, Feb 22, 2017 at 11:11 AM, Martin Jambor wrote: >> > Hello, >> > >> > this is a fix for PR 78140 which is about LTO WPA of

Re: [PATCH] Do not allow to run GIMPLE or RTL test-cases w/ LTO (PR lto/79625).

2017-02-28 Thread Richard Biener
On Tue, Feb 28, 2017 at 2:51 PM, Martin Liška wrote: > Hello. > > As mentioned in the PR, I hope rejecting -flto option with both GIMPLE and > RTL can > work for the issue. > > Ready after it finishes regression tests? Actually GIMPLE should work fine if you make sure to start

[PATCH,testsuite] Skip gcc.dg/lto/pr60449_0.c for mips*-*-elf* targets.

2017-02-28 Thread Toma Tabacu
Hi, The gcc.dg/lto/pr60449_0.c is failing for mips*-*-elf* targets because they do not support gettimeofday, which is used in this test case. This patch changes the test so that it is skipped for mips*-*-elf* targets. Regards, Toma gcc/testsuite/ * gcc.dg/lto/pr60449_0.c (dg-skip-if):

[PATCH, ARM] Fix PR79742 incorrect scheduler choice.

2017-02-28 Thread Richard Earnshaw (lists)
Due to an oversight, the changes to use the new CPU generation tables forgot to handle selecting a scheduler for a CPU other than the named CPU target. This meant that if, say, cortex-a12 was used, the null scheduler was chosen rather than cortex-a17's scheduler as intended. The fix is to

Re: [PR 78140] Reuse same IPA bits and VR info

2017-02-28 Thread Martin Jambor
Hi, On Mon, Feb 27, 2017 at 10:34:38AM +0100, Richard Biener wrote: > On Wed, Feb 22, 2017 at 11:11 AM, Martin Jambor wrote: > > Hello, > > > > this is a fix for PR 78140 which is about LTO WPA of Firefox taking > > 1GB memory more than gcc 6. > > > > It works by reusing the

[PATCH] Do not allow to run GIMPLE or RTL test-cases w/ LTO (PR lto/79625).

2017-02-28 Thread Martin Liška
Hello. As mentioned in the PR, I hope rejecting -flto option with both GIMPLE and RTL can work for the issue. Ready after it finishes regression tests? Thanks, Martin >From 3c26a29821b52d4e90fa05686a473b565ddf5ba9 Mon Sep 17 00:00:00 2001 From: marxin Date: Tue, 21 Feb 2017

maintainer-scripts/update_web_docs_svn

2017-02-28 Thread Gerald Pfeifer
While debugging an issue with our online documentation at https://gcc.gnu.org/onlinedocs (which I traced down to a bug with the old version of makeinfo on that host) I noticed WWWBASE was hardcoded without a way to override without changing the script. That's obviously not too helpful when

[PATCH 3/3] Introduce IntegerRange for options (PR driver/79659).

2017-02-28 Thread Martin Liška
Hi. Last patch from the mini-series fixes the PR mentioned in subject. I'm not sure whether it's stage4 material, or should I postpone it to next stage1? Thanks for review. Martin

[PATCH 2/3] Introduce = for couple of options.

2017-02-28 Thread Martin Liška
Hello. This is second patch, where I add '=' for some of options. Thanks, Martin >From add261f1fdf40b90d1a74dd7183923123943ba5e Mon Sep 17 00:00:00 2001 From: marxin Date: Mon, 27 Feb 2017 15:27:54 +0100 Subject: [PATCH 2/3] Introduce = for couple of options.

[PATCH] Add tabulars for options with enum values.

2017-02-28 Thread Martin Liška
Hello. We have various enum options that do not properly display options with --help=*: Apart from that, some of them are not separated by tabular, but with a space. Using the suggested patch, --help=common will get: --- /tmp/before 2017-02-28 10:41:12.107024945 +0100 +++ /tmp/after 2017-02-28

Re: PR79715: Special case strcpy/strncpy for dse

2017-02-28 Thread Prathamesh Kulkarni
On 28 February 2017 at 15:40, Jakub Jelinek wrote: > On Tue, Feb 28, 2017 at 03:33:11PM +0530, Prathamesh Kulkarni wrote: >> Hi, >> The attached patch adds special-casing for strcpy/strncpy to dse pass. >> Bootstrapped+tested on x86_64-unknown-linux-gnu. >> OK for GCC 8 ? > >

[PATCH][AArch64] Allow const0_rtx operand for atomic compare-exchange patterns

2017-02-28 Thread Kyrill Tkachov
Hi all, For the testcase in this patch we currently generate: foo: mov w1, 0 ldaxr w2, [x0] cmp w2, 3 bne .L2 stxrw3, w1, [x0] cmp w3, 0 .L2: csetw0, eq ret Note that the STXR could have been storing the

[FYI] Various SMS (modulo scheduling) patches

2017-02-28 Thread Roman Zhuykov
Hello, We at ISPRAS still have some SMS patches floating around from the time we have been actively working on them. At the time they were not committed because either of lack of interest or approval but they might be still useful as PR69252 discussion shows. We don’t have much resource now for

Re: [RFA PATCH, i386]: Warn for 64-bit values in general-reg asm operands and error out for 8-bit values in invalid GR asm operand

2017-02-28 Thread Jakub Jelinek
On Tue, Feb 28, 2017 at 11:41:56AM +0100, Richard Biener wrote: > > 2017-02-28 Uros Bizjak > > > > * config/i386/i386.c (print_reg): Warn for values of 64-bit size > > in integer register on 32-bit targets. Error out for values of > > 8-bit size in invalid

Re: [RFC PATCH 2/3] Use call_summary in ipa-prop and ipa-cp

2017-02-28 Thread jh
Dne 2017-02-27 17:35, Martin Jambor napsal: Hello, this is patch is afairly straightforward conversion from use of a vector indexed by edge->uid to use of the new call_summary from the previous patch. The patch is generally a cleanup, hashing is a nicer method of keeping call-site related

Re: [RFC PATCH 3/3] Remove ipa_update_after_lto_read

2017-02-28 Thread jh
Dne 2017-02-27 17:35, Martin Jambor napsal: Hello, when working on call summaries, I have found a weird function ipa_update_after_lto_read which currently only makes sure that IPA-CP function and edge summaries exist, at times when they already have to exist. So I did some digging in history

Re: [RFC PATCH 1/3] call_summary to keep info about cgraph_edges

2017-02-28 Thread jh
Dne 2017-02-27 17:35, Martin Jambor napsal: Hello, this patch is an actual implementation of the call_summary class (I hope the name is a good analogy to the function_summary we have, I am opened to other suggestions). I have kept the implementation close to the existing one of

Re: [RFA PATCH, i386]: Warn for 64-bit values in general-reg asm operands and error out for 8-bit values in invalid GR asm operand

2017-02-28 Thread Richard Biener
On Tue, 28 Feb 2017, Uros Bizjak wrote: > Hello! > > Attached patch: > > a) warns for 64-bit values in GR asm operand on 32bit targets > > It is impossible to pass 64-bit (long long) value in 32bit register. > We should warn for this situation, since only 32bit lowpart value is > passed. The

[RFA PATCH, i386]: Warn for 64-bit values in general-reg asm operands and error out for 8-bit values in invalid GR asm operand

2017-02-28 Thread Uros Bizjak
Hello! Attached patch: a) warns for 64-bit values in GR asm operand on 32bit targets It is impossible to pass 64-bit (long long) value in 32bit register. We should warn for this situation, since only 32bit lowpart value is passed. The warning can be omitted by explicitly casting asm operand to

Re: [BUILDROBOT] arm-netbsdelf: Error during -fself-test (was: [PATCH] TS_OPTIMIZATION/TS_TARGET_OPTION need no chain/type)

2017-02-28 Thread Jan-Benedict Glaw
On Mon, 2017-02-27 09:19:51 +0100, Richard Biener wrote: > On Mon, 27 Feb 2017, Jan-Benedict Glaw wrote: > > On Wed, 2017-01-11 16:28:33 +0100, Richard Biener wrote: > > > On Wed, 11 Jan 2017, Richard Biener wrote: > > > > LTO bootstrapped on

Re: [PR 78140] Reuse same IPA bits and VR info

2017-02-28 Thread jh
According to my measurements, the patch saves about 1.2 GB of memory. The problem is that some change last week (between revision 245382 and 245595) has more than invalidated this: | compiler| WPA mem (GB) | |-+--| | gcc 6 branch|

Re: PR79715: Special case strcpy/strncpy for dse

2017-02-28 Thread Jakub Jelinek
On Tue, Feb 28, 2017 at 03:33:11PM +0530, Prathamesh Kulkarni wrote: > Hi, > The attached patch adds special-casing for strcpy/strncpy to dse pass. > Bootstrapped+tested on x86_64-unknown-linux-gnu. > OK for GCC 8 ? What is special on strcpy/strncpy? Unlike memcpy/memmove/memset, you don't know

Re: Documentation patch related to PR42376

2017-02-28 Thread Gerald Pfeifer
On Mon, 27 Feb 2017, Sandra Loosemore wrote: > E I would have written "more slowly" (since this is an adverb > modifying "run"), but apparently both usages are correct. > > https://www.grammarly.com/answers/questions/79959-comparing-with-adverbs/ Thank you, Sandra! Since both are

[PATCH] Fix PR79731

2017-02-28 Thread Richard Biener
Bootstrap / regtest running on x86_64-unknown-linux-gnu. Richard. 2017-02-28 Richard Biener PR middle-end/79731 * fold-const.c (decode_field_reference): Reject out-of-bound accesses. * c-c++-common/torture/pr79731.c: New testcase. Index:

[PATCH] Fix PR79740

2017-02-28 Thread Richard Biener
Bootstrap / regtest running on x86_64-unknown-linux-gnu. Richard. 2017-02-28 Richard Biener PR tree-optimization/79740 * tree-ssa-sccvn.c (visit_nary_op): Insert the nary into the hashtable if we pattern-matched sth. * tree-ssa-pre.c

[PATCH] Fix PR79732

2017-02-28 Thread Richard Biener
Bootstrap / regtest running on x86_64-unknown-linux-gnu. Richard. 2017-02-28 Richard Biener PR tree-optimization/79732 * tree-inline.c (expand_call_inline): Do not shadow var. * gcc.dg/torture/pr79732.c: New testcase. Index: gcc/tree-inline.c

PR79715: Special case strcpy/strncpy for dse

2017-02-28 Thread Prathamesh Kulkarni
Hi, The attached patch adds special-casing for strcpy/strncpy to dse pass. Bootstrapped+tested on x86_64-unknown-linux-gnu. OK for GCC 8 ? Thanks, Prathamesh diff --git a/gcc/testsuite/gcc.dg/tree-ssa/pr79715.c b/gcc/testsuite/gcc.dg/tree-ssa/pr79715.c new file mode 100644 index 000..1a832ca

Re: [PATCH 0/6] Improve -fprefetch-loop-arrays in general and for AArch64 in particular

2017-02-28 Thread Maxim Kuvyrkov
> On Feb 20, 2017, at 5:38 PM, Kyrill Tkachov > wrote: > > Hi Maxim, > > On 30/01/17 11:24, Maxim Kuvyrkov wrote: >> This patch series improves -fprefetch-loop-arrays pass through small fixes >> and tweaks, and then enables it for several AArch64 cores. >> >> My

Re: [wwwdocs] Document in changes.html -fcode-hoisting, -fipa-bit-cp, -fipa-vrp, -fsplit-loops, GCJ removal, x86 ISA additions, -fshrink-wrap-separate etc.

2017-02-28 Thread Martin Jambor
Hi, On Mon, Feb 27, 2017 at 02:42:58PM -0800, Gerald Pfeifer wrote: > Hi Martin, > > at first I was surprised to see two suggestions from -- until I > realized those came from the two (Martins) of you. ;-) > > On Mon, 27 Feb 2017, Martin Jambor wrote: > > on top of your patch, I would like to

Re: [wwwdocs] Update changes.html for LTO and IPA

2017-02-28 Thread Jan Hubicka
> On Wed, 20 Jan 2016, Jan Hubicka wrote: > > this is updated patch. I tried to explain better the situation WRT > > incremental linking. > > Thank you, Jan. I had a couple of editorial changes on top of > this, which I finally managed to commit. (See the patch below.) > > And one question:

Miscellaneous optimization group fixes

2017-02-28 Thread Thomas Schwinge
Hi! On Wed, 22 Feb 2017 10:38:02 +0100, Martin Jambor wrote: > On Wed, Feb 22, 2017 at 08:58:06AM +0100, Thomas Schwinge wrote: > > On top of [...], OK to commit the following (not yet tested) -- these all > > look like oversights to me, but please verify? > > The missing

Rename the "openmp" group of optimizations to "omp" (was: Miscellaneous optimization group fixes)

2017-02-28 Thread Thomas Schwinge
Hi! On Wed, 22 Feb 2017 10:38:02 +0100, Martin Jambor wrote: > On Wed, Feb 22, 2017 at 08:58:06AM +0100, Thomas Schwinge wrote: > > > > > > Rename the "openmp" group of optimizations to "omp" > > > > > > gcc/ > > > * dumpfile.h