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

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

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 >>

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

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=gcc=rev Log: 2018-02-18 Jerry DeLisle PR fortran/84389 * io.c

[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

[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

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

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

[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

[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

[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

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

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