Re: [C++ Patch/RFC] PR 84348 ("[7/8 Regression] ICE with invalid friend declaration")

2018-02-18 Thread Jason Merrill
OK. On Fri, Feb 16, 2018 at 5:30 PM, Paolo Carlini wrote: > Hi, > > here we ICE during error recovery when, after emitting a correct error from > grokdeclarator, we go on, we only clear friendp, and grokfield proceeds to > call cp_finish_decl where 'gcc_assert (CLASS_PLACEHOLDER_TEMPLATE > (auto_

Re: [PATCH] replace ICE with error for failed template deduction (PR 84355)

2018-02-18 Thread Jason Merrill
On Fri, Feb 16, 2018 at 4:33 PM, Martin Sebor wrote: > On 02/16/2018 07:04 AM, Jason Merrill wrote: >> >> On Thu, Feb 15, 2018 at 6:36 PM, Martin Sebor wrote: >>> >>> A failed template deduction in template member of a template >>> triggers an ICE with -std=c++17 due to what seems like >>> a miss

Re: [PATCH] gold: Use autotools pthread macro

2018-02-18 Thread Joshua Watt
On Sat, Feb 17, 2018 at 4:42 PM, Cary Coutant wrote: >> The autotools library macro (AX_PTHREAD) is now used to detect if >> pthreads is present and multi-threaded linking in gold is automatically >> enabled if it is found. This enables multi-threaded gold on platforms >> where pthreads is enabled

Re: [Patch, fortran] PR83344 - Use of uninitialized memory with ASSOCIATE and strings

2018-02-18 Thread Paul Richard Thomas
Hi Janne and Thomas, 1) The patch is attached now - sorry! 2) The commented out part of associate_22.f90 is not yet fixed. I am working on it. 3) I will take a look at PR83975 tomorrow night. Paul On 18 February 2018 at 16:08, Janne Blomqvist wrote: > On Sun, Feb 18, 2018 at 5:48 PM, Paul Ri

New Spanish PO file for 'gcc' (version 8.1-b20180128)

2018-02-18 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 Spanish team of translators. The file is available at: http://translationproject.org/latest/gcc/es.po (This file, 'gcc-8.1-b20180128.es.po',

Re: [PATCH] Respect TMPDIR value in contrib scripts

2018-02-18 Thread Jeff Law
On 02/18/2018 11:25 AM, Yury Gribov wrote: > Hi all, > > This uses ${TMPDIR:-/tmp} instead of /tmp in scripts in contrib folder. > > Ok for trunk? > > -Y > OK. jeff

Re: [PATCHv2][PR 81376] Remove unnecessary float casts in comparisons

2018-02-18 Thread Jeff Law
On 02/18/2018 11:52 AM, Yuri Gribov wrote: > Hi all, > > This is a second iteration of patch which gets rid of float casts in > comparisons when all values of casted integral type are exactly > representable by the float type > (https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81376). The new version

[patch, committed] PR84389

2018-02-18 Thread Jerry DeLisle
Committed the following as obvious and simple to trunk with new test case. Author: jvdelisle Date: Sun Feb 18 19:19:47 2018 New Revision: 257795 URL: https://gcc.gnu.org/viewcvs?rev=257795&root=gcc&view=rev Log: 2018-02-18 Jerry DeLisle PR fortran/84389 * io.c (check_format):

[PATCHv2][PR 81376] Remove unnecessary float casts in comparisons

2018-02-18 Thread Yuri Gribov
Hi all, This is a second iteration of patch which gets rid of float casts in comparisons when all values of casted integral type are exactly representable by the float type (https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81376). The new version addresses Richard's review (https://gcc.gnu.org/ml/gcc-

[PATCH] Respect TMPDIR value in contrib scripts

2018-02-18 Thread Yury Gribov
Hi all, This uses ${TMPDIR:-/tmp} instead of /tmp in scripts in contrib folder. Ok for trunk? -Y Respect-TMPDIR-1.patch Description: Binary data

Re: [Fortran, PATCH, coarray, v1] Extend caf_*_by_ref () API by a type specifier

2018-02-18 Thread Andre Vehreschild
Well, after discussing on IRC whether RM should be bothered, I was asked to simplify release managers lives and propose, that if no one objects within one day, I will merge the patch. So any objections? - Andre On Sun, 18 Feb 2018 18:07:28 +0100 Andre Vehreschild wrote: > Dear release managers,

Re: [Fortran, PATCH, coarray, v1] Extend caf_*_by_ref () API by a type specifier

2018-02-18 Thread Andre Vehreschild
Dear release managers, this patch (for reference https://gcc.gnu.org/ml/fortran/2018-02/msg00124.html) fixes a regression in the coarray api by extending three relatively new functions with one or two arguments, respectively. The patch has been approved by gfortran devs. Asking your approval to me

Re: [Fortran, PATCH, coarray, v1] Extend caf_*_by_ref () API by a type specifier

2018-02-18 Thread Jerry DeLisle
On 02/18/2018 07:39 AM, Andre Vehreschild wrote: Hi all, attached patch fixes an issue with the coarray API. When a component of a derived type coarray was referenced using a caf_*_by_ref () function and that component was not an array with a descriptor, then the type of the component was not kn

[patch, fortran] Remove workaround introduced because of PR80945

2018-02-18 Thread Thomas Koenig
Hello world, after Paul's fix for PR80945, the code in frontend-passes.c meant to circumvent this bug is no longer needed. The attached patch removes it, adding a test case which shows that the optimization is working. After this, I think we can finally lay PR 35339 to rest. Regression-tested.

Re: [Patch, fortran] PR83344 - Use of uninitialized memory with ASSOCIATE and strings

2018-02-18 Thread Janne Blomqvist
On Sun, Feb 18, 2018 at 5:48 PM, Paul Richard Thomas wrote: > Bootstraps and regtests on FC27/x86_64 - OK for trunk? Hi, thanks for looking into this! 1. The patch itself is missing... 2. Could you uncomment the commented out part of associate_22.f90 and check that the tree-original dump is se

[Patch, fortran] PR83344 - Use of uninitialized memory with ASSOCIATE and strings

2018-02-18 Thread Paul Richard Thomas
Bootstraps and regtests on FC27/x86_64 - OK for trunk? Paul 2018-02-18 Paul Thomas PR fortran/83344 * resolve.c (resolve_assoc_var): Character associate names that have no length expression that have variable targets and are not deferred length have assumed length. * trans

[Fortran, PATCH, coarray, v1] Extend caf_*_by_ref () API by a type specifier

2018-02-18 Thread Andre Vehreschild
Hi all, attached patch fixes an issue with the coarray API. When a component of a derived type coarray was referenced using a caf_*_by_ref () function and that component was not an array with a descriptor, then the type of the component was not known. Which additionally meant, that type conversion

Re: [ patch, testsuite, fortran] Replace "call abort" by "stop n"

2018-02-18 Thread Janus Weil
2018-02-18 1:38 GMT+01:00 Thomas Koenig : > Hi Janus, > >> Regarding "-fall-intrinsics": Your commit has greatly reduced its >> usage, but I still see a few cases that you left in, although the flag >> does not really seem to be required. >> >> Is there a reason why did not treat these? > > I was s

New Swedish PO file for 'gcc' (version 8.1-b20180128)

2018-02-18 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 Swedish team of translators. The file is available at: http://translationproject.org/latest/gcc/sv.po (This file, 'gcc-8.1-b20180128.sv.po',

[Patch, fortran] PR80945 - Invalid code with allocatable character array in READ/WRITE statement

2018-02-18 Thread Paul Richard Thomas
Committed as 'obvious' - revision 257788. This patch works fine on 7-branch. I will commit it there in a day or two unless there is any objection. Cheers Paul 2018-02-18 Paul Thomas PR fortran/80945 * trans-array.c (gfc_conv_expr_descriptor): Set parmtype from the typenode in th