[PATCH][PR middle-end/79521] Refine condition for calling ira_init_register_move_cost_if_necessary

ira_init_register_move_cost_if_necessary asserts have_regs_of_mode[MODE] is true. We need to make sure not to call ira_init_register_move_cost_if_necessary when have_regs_of_mode[MODE] is false. Verified the H8 port still builds libgcc, also bootstrapped and regression tested on

[patch, contrib] Add support to install libcaf-mpi for multi-image coarray Fortran

The attached patch adds a new subdirectory called mk-libcaf-multi under contrib which contains scripts which will download OpenCoarrays, build libcaf-mpi.a, and install it in the user provided --install-prefix. As given the script is only manually executed by a user interested in doing so.

Re: [libstdc++,doc] Strip links to ANSI (web shop)

On 09/02/17 15:15 +0100, Gerald Pfeifer wrote: On Sun, 5 Feb 2017, Jonathan Wakely wrote: ANSI sells the standard for a much more reasonable price than ISO or any of the other national standards bodies. I think many people use draft standards now, rather than buying it, but for those who do

Re: PR rtl-optimization/64081: Enable RTL loop unrolling for duplicated exit blocks and back edges (with AIX fixes)

On 02/13/2017 07:15 PM, Jeff Law wrote: So it seems in your updated patch there is only one call where we ask for LOOP_EXIT_COMPLEX, specifically the call from get_loop_location. But I don't see how asking for LOOP_EXIT_COMPLEX from that location would change whether or not we unroll any given

Re: [PATCH v5] add -fprolog-pad=N,M option

On 15/02/17 11:12, Marek Polacek wrote: > On Wed, Feb 15, 2017 at 11:01:16AM +, Richard Earnshaw (lists) wrote: >> On 13/01/17 12:19, Torsten Duwe wrote: >>> Changes since v4: hopefully addressed all of Sandra's requests >>> and suggestions concerning the documentation snippets, thanks >>> for

[PATCH] PR 68749: S/390: Disable ifcvt-4.c for -m31.

The attached patch disables the test ifcvt-4.c on s390 and on s390x with -31, and adds -march=z196 for s390x. It should no longer fail on s390 and s390x. Tested on s390x biarch. Ciao Dominik ^_^ ^_^ -- Dominik Vogt IBM Germany gcc/testsuite/ChangeLog-pr68749 PR 68749 *

Re: [RFC PATCH] Improve switchconv optimization (PR tree-optimization/79472)

On 15/02/17 08:17, Jakub Jelinek wrote: > On Wed, Feb 15, 2017 at 08:06:16AM +0100, Richard Biener wrote: >> On February 14, 2017 9:04:45 PM GMT+01:00, Jakub Jelinek >> wrote: >>> Hi! >>> >>> The following patch is an attempt to fix a regression where we no >>> longer >>>

[GIMPLE FE] avoid ICE with same ssa version number for multiple names

Hi, For the following (invalid) test-case: void __GIMPLE () foo (int a) { int t0; int _1; _1 = a; t0_1 = a; } results in following ICE: gimplefe-error-4.c: In function ‘foo’: gimplefe-error-4.c:20:1: error: SSA_NAME_DEF_STMT is wrong } ^ Expected definition statement: _1 = a_2(D);

[PATCH][GCC6] Backport PR target/76731 fix

Hi, Is it ok to backport this fix to GCC6 branch? PR target/76731 * config/i386/avx512fintrin.h (_mm512_i32gather_ps): Change __addr type to void const*. (_mm512_mask_i32gather_ps): Ditto. (_mm512_i32gather_pd): Ditto. (_mm512_mask_i32gather_pd):

Re: [PATCH v5] add -fprolog-pad=N,M option

On 13/01/17 12:19, Torsten Duwe wrote: > Changes since v4: hopefully addressed all of Sandra's requests > and suggestions concerning the documentation snippets, thanks > for the feedback. If it still isn't clear, feel free to rephrase > -- I'm a programmer, not a technical writer. > Generally,

Re: [PATCH v5] add -fprolog-pad=N,M option

On Wed, Feb 15, 2017 at 11:01:16AM +, Richard Earnshaw (lists) wrote: > On 13/01/17 12:19, Torsten Duwe wrote: > > Changes since v4: hopefully addressed all of Sandra's requests > > and suggestions concerning the documentation snippets, thanks > > for the feedback. If it still isn't clear,

Re: [PATCH, wwwdocs/ARM] Deprecate ARMv5 and ARMv5E support

On 15/02/17 11:23, Thomas Preudhomme wrote: > Hi, > > ARMv5 and ARMv5E architectures have no known implementation. I therefore > suggest that we deprecate these architectures. ARMv5T, ARMv5TE and > ARMv5TEJ would remain supported though. > > Is this ok to commit? > > Best regards, > > Thomas >

C PATCH to fix ICE with -Wdouble-promotion (PR c/79515)

We ICEd on this testcase in do_warn_double_promotion because an invalid conversion had produced an error result type and accessing that via TYPE_MAIN_VARIANT crashes. Fixed in an obvious way. Bootstrapped/regtested on x86_64-linux, ok for trunk? 2017-02-15 Marek Polacek

Re: [PATCH] Fix DFP conversion from INTEGER_CST to REAL_CST (PR target/79487)

On Wed, Feb 15, 2017 at 09:15:48AM +0100, Richard Biener wrote: > >As the following testcase shows, we store decimal REAL_CSTs always in > >_Decimal128 internal form and perform all the arithmetics on that, but > >while > >for arithmetics we then ensure rounding to the actual type >

Re: [PATCH][GRAPHITE] Remove support for ISL 0.14

Hi! On Fri, 10 Feb 2017 15:13:57 +0100 (CET), Richard Biener wrote: > As a cleanup (and to be able to close bugs only reproducing with ISL 0.14) > the following removes support for ISL 0.14 for GCC 7. (This got committed in r245382.) > --- config/isl.m4 (revision

[PATCH, wwwdocs/ARM] Deprecate ARMv5 and ARMv5E support

Hi, ARMv5 and ARMv5E architectures have no known implementation. I therefore suggest that we deprecate these architectures. ARMv5T, ARMv5TE and ARMv5TEJ would remain supported though. Is this ok to commit? Best regards, Thomas cvs diff: Diffing . cvs diff: Diffing benchmarks cvs diff:

Re: [RFC PATCH] Improve switchconv optimization (PR tree-optimization/79472)

On Wed, Feb 15, 2017 at 12:46:44PM +0100, Richard Biener wrote: > >> Possibly, but for GCC 8. > > > > To both this switchconv patch and the potential improvement for loading > > from const arrays (can create an enhancement PR for that), or just the > > latter? > > Both I think - the patch is

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

Hi! On Wed, 1 Feb 2017 20:26:24 -0700, Martin Sebor wrote: > On 02/01/2017 08:06 PM, Sandra Loosemore wrote: > > On 02/01/2017 06:57 PM, Martin Sebor wrote: > PR middle-end/32003 > * doc/invoke.texi (-fdump-rtl-): Remove pass-specific options from > index.

Re: [PATCH] Fix DFP conversion from INTEGER_CST to REAL_CST (PR target/79487)

On February 15, 2017 7:53:58 AM GMT+01:00, Jakub Jelinek wrote: >Hi! > >As the following testcase shows, we store decimal REAL_CSTs always in >_Decimal128 internal form and perform all the arithmetics on that, but >while >for arithmetics we then ensure rounding to the actual

Re: [v3 PATCH] Implement C++17 GB50 resolution

Hi, Dinka, thanks for the patch. On 14/02/17 21:22 +, Dinka Ranns wrote: diff --git a/libstdc++-v3/include/std/chrono b/libstdc++-v3/include/std/chrono index ceae7f8..6a6995c 100644 --- a/libstdc++-v3/include/std/chrono +++ b/libstdc++-v3/include/std/chrono @@ -349,50 +349,50 @@

Re: [PATCH] PR target/79241: S/390: define TARGET_CUSTOM_FUNCTION_DESCRIPTORS.

On 02/09/2017 07:16 PM, Dominik Vogt wrote: > The attached patch fixes PR 79241 on s390x. Bootstrapped and > regression tested on s390x biarch (not tested on s390). > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79421 > > Ciao > > Dominik ^_^ ^_^ > Applied. Thanks! -Andreas-

Re: [PATCH] Implement P0393R3

On Mon, Jan 9, 2017 at 2:52 AM, Jonathan Wakely wrote: > On 08/01/17 22:49 -0800, Tim Shen wrote: >> >> On Tue, Jan 3, 2017 at 6:17 AM, Jonathan Wakely >> wrote: >>> >>> On 01/01/17 04:17 -0800, Tim Shen via libstdc++ wrote: +#define

[PATCH, Fortran, pr79335, v2] [7 Regression] Conditional jump or move depends on uninitialised in value get_scalar_to_descriptor_type(tree_node*, symbol_attribute) (trans-expr.c:53)

Hi all, attached patch fixes (hopefully) all occurrences of uninitialized symbol_attributes used for getting a descriptor for a scalar as reported in the reopended pr79335. Bootstraps and regtests ok on x86_64-linux/f25. Ok for trunk? Regards, Andre On Tue, 14 Feb 2017 08:32:59 +

[PATCH] Enable RDPID instruction.

Hi, This patch enables RDPID intrinsic, described in SDM 4-534 Vol. 2B https://software.intel.com/sites/default/files/managed/39/c5/325462-sdm-vol-1-2abcd-3abcd.pdf and intrinsic guide: https://software.intel.com/sites/landingpage/IntrinsicsGuide/#text=rdpi=2778,2777,4219 gcc/ *

[PATCH] PR 79356: Do not xfail attr-alloc_size-11.c on some targets.

The attached patch removes the xfail on two sub tests of attr-alloc_size-11.c for the targets who were reported to produce an XPASS: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79356 Tested on s390x biarch and s390. Please check if the strings for the other targets are correct. Ciao Dominik

Re: [RFC PATCH] Improve switchconv optimization (PR tree-optimization/79472)

On 02/15/2017 04:51 AM, Jakub Jelinek wrote: On Wed, Feb 15, 2017 at 12:46:44PM +0100, Richard Biener wrote: Possibly, but for GCC 8. To both this switchconv patch and the potential improvement for loading from const arrays (can create an enhancement PR for that), or just the latter? Both I

Re: Handle GIMPLE NOPs in is_maybe_undefined (PR, tree-optimization/79529).

On 02/15/2017 09:49 AM, Martin Liška wrote: Hi. As mentioned in the PR, gimple nops are wrongly handled in is_maybe_undefined function. Patch can bootstrap on ppc64le-redhat-linux and survives regression tests. gimple *def = SSA_NAME_DEF_STMT (t); + if (!def || gimple_nop_p

[libgomp, testsuite] Enable libgomp.c/pr48591.c on __float128 targets

When comparing x86_64-pc-linux-gnu and i386-pc-solaris2.12 testsuite results for PR rtl-optimization/79532, I noticed that libgomp.c/pr48591.c wasn't run on Solaris/x86. The testcase contains a seemingly random list of targets to run on, but ISTM that it should really check for __float128 support

Re: [libgomp, testsuite] Enable libgomp.c/pr48591.c on __float128 targets

On Wed, Feb 15, 2017 at 05:28:48PM +0100, Rainer Orth wrote: > When comparing x86_64-pc-linux-gnu and i386-pc-solaris2.12 testsuite > results for PR rtl-optimization/79532, I noticed that > libgomp.c/pr48591.c wasn't run on Solaris/x86. The testcase contains a > seemingly random list of targets

C++ PATCH for c++/79464 (ICE in IPA with inherited ctor)

In the new inherited constructor framework, base inheriting constructor variants don't retain the parameters inherited from a virtual base constructor, since that constructor won't actually be called from there; it will be called directly by the complete object constructor. Previously, I had cut

Re: [PATCH][RFA][target/79404] Fix uninitialized reference to ira_register_move_cost[mode]

On 02/15/2017 12:42 PM, Gerald Pfeifer wrote: Hi Jeff, On Mon, 13 Feb 2017, Jeff Law wrote: I've verified this allows libgcc to build on the H8 target and bootstrapped/regression tested the change on x86_64-unknown-linux-gnu as well. I need to reproduce this, but my latest daily bootstrap on

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

On 02/15/2017 05:51 AM, Thomas Schwinge wrote: Hi! On Wed, 1 Feb 2017 20:26:24 -0700, Martin Sebor wrote: On 02/01/2017 08:06 PM, Sandra Loosemore wrote: On 02/01/2017 06:57 PM, Martin Sebor wrote: PR middle-end/32003 * doc/invoke.texi (-fdump-rtl-): Remove

Re: [C++ RFC] Fix up attribute handling in templates (PR c++/79502)

OK. On Wed, Feb 15, 2017 at 12:30 PM, Jakub Jelinek wrote: > On Wed, Feb 15, 2017 at 11:56:30AM -0500, Jason Merrill wrote: >> On Tue, Feb 14, 2017 at 3:13 PM, Jakub Jelinek wrote: >> > The following testcase fails, because while we have the nodiscard >> >

[PATCH v4] aarch64: Add split-stack initial support

This is an update patch from my previous version (v3) based mainly on glibc comments on exported loader symbol [1]. The changes from previous version are: - Remove __tcb_private_ss call on libgcc and emit a data usage on glibc symbol when split-stack is used. This removes the runtime

Re: C PATCH to fix ICE with -Wdouble-promotion (PR c/79515)

On 02/15/2017 12:49 PM, Marek Polacek wrote: We ICEd on this testcase in do_warn_double_promotion because an invalid conversion had produced an error result type and accessing that via TYPE_MAIN_VARIANT crashes. Fixed in an obvious way. Bootstrapped/regtested on x86_64-linux, ok for trunk?

[Patch, fortran] PR79434 - [submodules] separate module procedure breaks encapsulation

Dear All, Another straightforward patch, although it took some head scratching to realize how simple the fix could be :-) Bootstraps and regtests on FC23/x_86_64 - OK for trunk and 6-branch? Cheers Paul 2017-02-15 Paul Thomas PR fortran/79434 * parse.c

[Patch, fortran] PR79447 - [F08] gfortran rejects valid & accepts invalid internal subprogram in a submodule

Dear All, This patch is straightforward, verging on 'obvious'. Bootstraps and regtests on FC23/x86_64 - OK for trunk and 6 branch? Cheers Paul 2017-02-15 Paul Thomas PR fortran/79447 * decl.c (gfc_set_constant_character_len): Whitespace. (gfc_match_end):

Re: [C++ RFC] Fix up attribute handling in templates (PR c++/79502)

On Wed, Feb 15, 2017 at 11:56:30AM -0500, Jason Merrill wrote: > On Tue, Feb 14, 2017 at 3:13 PM, Jakub Jelinek wrote: > > The following testcase fails, because while we have the nodiscard > > attribute on the template, we actually never propagate it to the > > instantiation,

Re: [PATCH][AArch64] Accept more addressing modes for PRFM

On 15/02/17 15:03, Kyrill Tkachov wrote: > Hi Richard, > > On 15/02/17 15:00, Richard Earnshaw (lists) wrote: >> On 03/02/17 17:12, Kyrill Tkachov wrote: >>> Hi all, >>> >>> While evaluating Maxim's SW prefetch patches [1] I noticed that the >>> aarch64 prefetch pattern is >>> overly restrictive

Re: [C++ PATCH] Don't pedwarn on deprecated or fallthrough attributes (PR c++/79301)

OK. On Fri, Feb 10, 2017 at 2:53 PM, Jakub Jelinek wrote: > Hi! > > The reporter complained that even when __has_cpp_attribute (fallthrough) > and similarly __has_cpp_attribute (deprecated) return true, in pedantic > mode the compiler will error out on those. > > The following

Re: [AArch64] Accelerate -fstack-protector through pointer authentication extension

On 18/01/17 17:10, Jiong Wang wrote: >> NOTE, this approach however requires DWARF change as the original LR >> is signed, >> the binary needs new libgcc to make sure c++ eh works correctly. >> Given this >> acceleration already needs the user specify >> -mstack-protector-dialect=pauth >> which

Re: [PATCH] avoid ICE when attempting to init a flexible array member (PR c++/79363)

OK. On Mon, Feb 6, 2017 at 9:04 PM, Martin Sebor wrote: > The attached patch avoids another ICE (in addition to the already > fixed bug 72775) in flexible array member NSDMI. To avoid code > duplication and for consistency I factored the diagnostic code > out of

Re: [PATCH PR79347]Maintain profile counter information in vect_do_peeling

> On Tue, Feb 14, 2017 at 2:13 PM, Bin.Cheng wrote: > > On Tue, Feb 14, 2017 at 1:57 PM, Jan Hubicka wrote: > >>> Thanks, > >>> bin > >>> 2017-02-13 Bin Cheng > >>> > >>> PR tree-optimization/79347 > >>> *

Re: Patch ping^2

On Tue, Feb 14, 2017 at 10:05:04AM -0500, Nathan Sidwell wrote: > On 02/13/2017 10:46 AM, Jakub Jelinek wrote: > > Hi! > > > > I'd like to ping a couple of patches: > > > - C++ P1 PR79288 - wrong default TLS model for __thread static data members > >

Re: [C++ RFC] Fix up attribute handling in templates (PR c++/79502)

On Tue, Feb 14, 2017 at 3:13 PM, Jakub Jelinek wrote: > The following testcase fails, because while we have the nodiscard > attribute on the template, we actually never propagate it to the > instantiation, which is where it is checked (I'm really surprised about > this).

Re: [gomp4] adjust num_gangs and add a diagnostic for unsupported num_workers

Hi Cesar! On Mon, 13 Feb 2017 08:58:39 -0800, Cesar Philippidis wrote: > This patch does the followings: > > * Adjusts the default num_gangs to utilize more of GPU hardware. > * Teach libgomp to emit a diagnostic when num_workers isn't supported. > > [...] Thanks!

Re: [PATCH][RFA][target/79404] Fix uninitialized reference to ira_register_move_cost[mode]

Hi Jeff, On Mon, 13 Feb 2017, Jeff Law wrote: I've verified this allows libgcc to build on the H8 target and bootstrapped/regression tested the change on x86_64-unknown-linux-gnu as well. I need to reproduce this, but my latest daily bootstrap on i386-unknown-freebsd10.3 failed with...

New Spanish PO file for 'gcc' (version 7.1-b20170101)

Hello, gentle maintainer. This is a message from the Translation Project robot. A revised PO file for textual domain 'gcc' has been submitted by the Spanish team of translators. The file is available at: http://translationproject.org/latest/gcc/es.po (This file, 'gcc-7.1-b20170101.es.po',

Re: [gomp4] Async related additions to OpenACC runtime library

Hi! On Tue, 14 Feb 2017 19:58:11 +0800, Chung-Lin Tang wrote: > On 2017/2/14 07:25 PM, Thomas Schwinge wrote: > > Testing [...], I saw a lot of regressions, > > and in r245427 just committed [...] to address > > these. Did you simply forget to commit your changes to >

Re: [AArch64] Accelerate -fstack-protector through pointer authentication extension

On 15/02/17 15:45, Richard Earnshaw (lists) wrote: On 18/01/17 17:10, Jiong Wang wrote: NOTE, this approach however requires DWARF change as the original LR is signed, the binary needs new libgcc to make sure c++ eh works correctly. Given this acceleration already needs the user specify

Re: [Patch, fortran] PR79447 - [F08] gfortran rejects valid & accepts invalid internal subprogram in a submodule

On 02/15/2017 10:01 AM, Paul Richard Thomas wrote: > Dear All, > > This patch is straightforward, verging on 'obvious'. > > Bootstraps and regtests on FC23/x86_64 - OK for trunk and 6 branch? > Yes, OK Jerry

Re: [Patch, fortran] PR79434 - [submodules] separate module procedure breaks encapsulation

On 02/15/2017 10:12 AM, Paul Richard Thomas wrote: > Dear All, > > Another straightforward patch, although it took some head scratching > to realize how simple the fix could be :-) > > Bootstraps and regtests on FC23/x_86_64 - OK for trunk and 6-branch? > Yes, OK Jerry

Re: [PATCH][GRAPHITE] Remove support for ISL 0.14

On February 15, 2017 1:44:13 PM GMT+01:00, Thomas Schwinge wrote: >Hi! > >On Fri, 10 Feb 2017 15:13:57 +0100 (CET), Richard Biener > wrote: >> As a cleanup (and to be able to close bugs only reproducing with ISL >0.14) >> the following removes support

Re: [PATCH][GRAPHITE] Remove support for ISL 0.14

On Wed, Feb 15, 2017 at 6:44 AM, Thomas Schwinge wrote: > Hi! > > On Fri, 10 Feb 2017 15:13:57 +0100 (CET), Richard Biener > wrote: >> As a cleanup (and to be able to close bugs only reproducing with ISL 0.14) >> the following removes support for ISL

Re: [PATCH][GRAPHITE] Remove support for ISL 0.14

Hi! On Wed, 15 Feb 2017 13:44:13 +0100, I wrote: > On Fri, 10 Feb 2017 15:13:57 +0100 (CET), Richard Biener > wrote: > > As a cleanup (and to be able to close bugs only reproducing with ISL 0.14) > > the following removes support for ISL 0.14 for GCC 7. > OK to commit the

Handle GIMPLE NOPs in is_maybe_undefined (PR, tree-optimization/79529).

Hi. As mentioned in the PR, gimple nops are wrongly handled in is_maybe_undefined function. Patch can bootstrap on ppc64le-redhat-linux and survives regression tests. Ready to be installed? Martin >From 54b98e2d035f92ec20bf7b548f90b1d00c4c597b Mon Sep 17 00:00:00 2001 From: marxin

Re: [PATCH][AArch64] Accept more addressing modes for PRFM

On 03/02/17 17:12, Kyrill Tkachov wrote: > Hi all, > > While evaluating Maxim's SW prefetch patches [1] I noticed that the > aarch64 prefetch pattern is > overly restrictive in its address operand. It only accepts simple > register addressing modes. > In fact, the PRFM instruction accepts almost

Re: [PATCH][AArch64] Accept more addressing modes for PRFM

Hi Richard, On 15/02/17 15:00, Richard Earnshaw (lists) wrote: On 03/02/17 17:12, Kyrill Tkachov wrote: Hi all, While evaluating Maxim's SW prefetch patches [1] I noticed that the aarch64 prefetch pattern is overly restrictive in its address operand. It only accepts simple register addressing

Re: [PATCH PR79347]Maintain profile counter information in vect_do_peeling

On Tue, Feb 14, 2017 at 2:13 PM, Bin.Cheng wrote: > On Tue, Feb 14, 2017 at 1:57 PM, Jan Hubicka wrote: >>> Thanks, >>> bin >>> 2017-02-13 Bin Cheng >>> >>> PR tree-optimization/79347 >>> * tree-vect-loop-manip.c