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

2017-02-19 Thread Paul Richard Thomas
Hi Jerry, Committed as revision 245582. Thanks for the review. Paul On 16 February 2017 at 00:37, Jerry DeLisle wrote: > On 02/15/2017 10:01 AM, Paul Richard Thomas wrote: >> Dear All, >> >> This patch is straightforward, verging on 'obvious'. >> >> Bootstraps and

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

2017-02-19 Thread Martin Sebor
On 02/17/2017 09:53 PM, Jason Merrill wrote: On Thu, Feb 16, 2017 at 6:13 PM, Martin Sebor wrote: On 02/16/2017 12:49 PM, Jason Merrill wrote: On Thu, Feb 16, 2017 at 11:33 AM, Jakub Jelinek wrote: PR c++/79502 * pt.c

Re: Ping [Patch, fortran] PR79402 - ICE with submodules: module procedure interface defined in parent module

2017-02-19 Thread Paul Richard Thomas
Hi Everybody, With Jerry's OK for a commit to trunk, committed as revision 245580. I will wait a few weeks before a commit to 6-branch. Cheers Paul On 11 February 2017 at 13:24, Paul Richard Thomas wrote: > Ping! > > On 8 February 2017 at 16:00, Paul Richard

Re: [Ping, PATCH, Fortran, 79229, v1] [7 Regression] ICE in gfc_trans_assignment_1 with -fcheck=mem

2017-02-19 Thread Andre Vehreschild
Hi Jerry, hi all, thanks for the review. Committed as r245581. Regards, Andre On Sun, 19 Feb 2017 08:58:46 -0800 Jerry DeLisle wrote: > On 02/19/2017 08:42 AM, Andre Vehreschild wrote: > > Ping! > > > > this time with correct PR number: 79229. > > > > On Tue, 7

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

2017-02-19 Thread Translation Project Robot
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 German team of translators. The file is available at: http://translationproject.org/latest/gcc/de.po (This file, 'gcc-7.1-b20170101.de.po',

[wwwdocs] projects/cfg.html - use doi.org instead of citeseer.ist.psu.edu

2017-02-19 Thread Gerald Pfeifer
citeseer has been acting interestingly for a while, and now is blocking clients based on geography (verified as blocking from three different systems/networks in Austria and Germany on the one hand¹, and working from the gcc.gnu.org host and another one in North America on the other). So,

[libstdc++,doc] doc/xml/manual/debug.xml: Adjust link to ThreadSanitizer

2017-02-19 Thread Gerald Pfeifer
The site at code.google.com redirects to github now. Applied. Gerald 2017-02-19 Gerald Pfeifer * doc/xml/manual/debug.xml: Adjust link to ThreadSanitizer. Index: doc/xml/manual/debug.xml === ---

[wwwdocs] PATCH Re: DWARF Version 5 Standard Released

2017-02-19 Thread Gerald Pfeifer
On Wed, 15 Feb 2017, Michael Eager wrote: > The DWARF Debugging Information Format Standards Committee is pleased > to announce the availability of Version 5 of the DWARF Debugging Format > Standard. This patch, which I just applied, updates the GCC 7 release notes accordingly. (Now it's only

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-19 Thread Gerald Pfeifer
That was quite a bit; thanks for doing that, Jakub! In the patch below I try to streamline language a bit, document options being implied by -Os (in addition to -O2 or higher), fix grammar in a few places, and change a textual link to being a real one. I have _not_ applied this yet, and would

[v3 PATCH] Make optional's comparisons be two-parameter templates.

2017-02-19 Thread Ville Voutilainen
This has not been adopted by LEWG/LWG yet, but was submitted as a proposed resolution for a new LWG issue; optional can't current be compared to a T, and an optional can't be compared to something non-T that is comparable to a T. This approach fixes both of those problems, allowing optional's

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

2017-02-19 Thread Jonathan Wakely
On 18/02/17 20:48 +, Dinka Ranns wrote: Comments addressed. Please find the new diff attached to this e-mail. Changelog after review comments : 2017-02-18 Dinka Ranns GNU ChangeLog rules say two spaces after the date and after your name. C++17 GB50

Re: [Ping, PATCH, Fortran, 79229, v1] [7 Regression] ICE in gfc_trans_assignment_1 with -fcheck=mem

2017-02-19 Thread Andre Vehreschild
Ping! this time with correct PR number: 79229. On Tue, 7 Feb 2017 08:31:22 +0100 Andre Vehreschild wrote: > Hi all, > > attached patch fixes the issue of 79229 (which is not a regression). > The issue was that the code generating the checking was expecting a > pointer type, but

Re: [Ping, PATCH, Fortran, 79229, v1] [7 Regression] ICE in gfc_trans_assignment_1 with -fcheck=mem

2017-02-19 Thread Jerry DeLisle
On 02/19/2017 08:42 AM, Andre Vehreschild wrote: Ping! this time with correct PR number: 79229. On Tue, 7 Feb 2017 08:31:22 +0100 Andre Vehreschild wrote: Hi all, attached patch fixes the issue of 79229 (which is not a regression). The issue was that the code generating the

Re: gcc.css colors

2017-02-19 Thread Gerald Pfeifer
Hi Markus, On Wed, 1 Feb 2017, Markus Trippelsdorf wrote: > Some colors on e.g. https://gcc.gnu.org/gcc-7/changes.html are nearly > unreadable. So what about the following patch? > > -.boldcyan{ font-weight:bold; color:cyan; } > -.boldlime{ font-weight:bold; color:lime; } > +.boldcyan

Re: [RFC PATCH, i386]: Use "lock orl $0, -4(%esp)" in mfence_nosse

2017-02-19 Thread Uros Bizjak
On Fri, Feb 17, 2017 at 5:59 PM, Uros Bizjak wrote: > On Fri, Feb 17, 2017 at 5:30 PM, Jakub Jelinek wrote: >> On Sun, May 29, 2016 at 11:10:15PM +0200, Uros Bizjak wrote: >>> As explained in PR71245, comment #3 [1], it is better to use offset -4 >>> to a

C++ PATCH for c++/79500, ICE with non-template deduction guide

2017-02-19 Thread Jason Merrill
DECL_TEMPLATE_RESULT doesn't work on a non-template, so let's use STRIP_TEMPLATE instead. Tested x86_64-pc-linux-gnu, applying to trunk. commit 9f99104a12b72006253c0ce5f0b046ada82eb354 Author: Jason Merrill Date: Sun Feb 19 14:01:51 2017 -0800 PR c++/79500 - ICE

C++ PATCH for c++/79400, confusing suggestion of 'noexcept'

2017-02-19 Thread Jason Merrill
It was pointed out that suggesting noexcept as an alternative to throw(type) doesn't make much sense, since throw(type) would imply noexcept(false), which is the default. So let's just remove the suggestion. Tested x86_64-pc-linux-gnu, applying to trunk. commit

C++ PATCH for c++/79566, elaborated type-specifier in range for

2017-02-19 Thread Jason Merrill
In the parser, declares_class_or_enum is set to 1 for an elaborated type-specifier, or 2 for a class or enum definition. We only want to complain about a type definition in the latter case. Tested x86_64-pc-linux-gnu, applying to trunk. commit 8b366b7f80be1f160bd31501aa436e4d300cd753 Author:

C++ PATCH for c++/79607, ICE with T{} initializer

2017-02-19 Thread Jason Merrill
type_dependent_init_p was overlooking the possibility of a CONSTRUCTOR with a dependent type, which we can get from T{}. Tested x86_64-pc-linux-gnu, applying to trunk. commit 060f95851e7f6d6737937b786c0a6b29cda3c454 Author: Jason Merrill Date: Sun Feb 19 15:40:39 2017 -0800

C++ PATCH for c++/79606, ICE with this->base_member in NSDMI

2017-02-19 Thread Jason Merrill
build_base_path knows that it should limit what it tries to do within a template, but that handling wasn't being used within an NSDMI, only within a template function. Tested x86_64-pc-linux-gnu, applying to trunk. commit 7a11cba1268aa91df1bc253c43deb2dcbfd5b091 Author: Jason Merrill

Ping Re: [patch, doc] copy-edit ARC options documentation

2017-02-19 Thread Sandra Loosemore
On 02/13/2017 11:14 PM, Sandra Loosemore wrote: 2017-02-13 Sandra Loosemore gcc/ * doc/invoke.texi (ARC Options): Copy-edit to fix punctuation, markup, and similar issues. Remove @opindex entries for things that aren't options.

C++ PATCH for c++/79503 (inherited ctor taking base class)

2017-02-19 Thread Jason Merrill
In the old inheriting constructor scheme, copy constructors were not inherited. In the new scheme we needed a rule to get a similar effect; I proposed that an inherited constructor should not be viable if it has a single parameter of a type reference-related to the derived type. Richard Smith

C++ PATCH for c++/79580, ICE with compound literal

2017-02-19 Thread Jason Merrill
We were treating a compound literal type defined in the initializer for a static data member as though it were a member of the class. Fixed by ignoring classes when defining a type from within an expression. Tested x86_64-pc-linux-gnu, applying to trunk. commit

C++ PATCH for c++/79470, partial ordering with reference parameters

2017-02-19 Thread Jason Merrill
We have code in unify to handle the implicit indirection through a reference, but it couldn't handle when that indirection was wrapped in an EXPR_PACK_EXPANSION. Tested x86_64-pc-linux-gnu, applying to trunk. commit 75e6ade977ea2d229d5ccd4fdaaf812c634c4d41 Author: Jason Merrill

C++ PATCH for c++/78282, auto template and pack expansion

2017-02-19 Thread Jason Merrill
When looking for parameter packs mentioned within an expression, we were wrongly walking into the type of a TEMPLATE_DECL mentioned in the expression. We're only interested in packs in the type of a template template-parameter, not other kinds of template. Tested x86_64-pc-linux-gnu, applying to