[patch, testsuite] make g++.dg/ipa/pr89009.C require fpic effective target

2019-04-20 Thread Sandra Loosemore
I've checked in the attached patch as obvious, to clean up some test failures on nios2-elf. (The nios2 PSABI only defines PIC relocations for GNU/Linux targets so gcc rejects -fpic when compiling for bare-metal target.) -Sandra 2019-04-20 Sandra Loosemore gcc/testsuite/ *

Re: [PATCH] Delegate PSTL configuration to pstl/pstl_config.h

2019-04-20 Thread Thomas Rodgers
Jonathan Wakely writes: > On 18/04/19 17:02 -0700, Thomas Rodgers wrote: >> >> * include/bits/c++config: Remove explicit PSTL configuration >> macros and use definitions from . > > OK for trunk, thanks. Tested x86_64-linux-gnu, committed to trunk.

Re: [PATCH] Cleanup algorithm implementations

2019-04-20 Thread Thomas Rodgers
Jonathan Wakely writes: > On 19/04/19 11:59 -0700, Thomas Rodgers wrote: >> * include/pstl/glue_algorithm_impl.h (stable_sort): Forward >>execution policy. >> (mismatch): Forward execution policy. >> (equal): Qualify call to std::equal(). >> (partial_sort): Forward

Re: [PATCH] Improve implementation of parallel equal()

2019-04-20 Thread Thomas Rodgers
Jonathan Wakely writes: > On 16/04/19 12:39 -0700, Thomas Rodgers wrote: >> >> * include/pstl/algorithm_impl.h >> (__internal::__brick_equal): use "4 iterator" version of >> std::equal(). >> (__internal::__brick_equal): use simd for random access >> iterators on

Re: [PATCH, LRA]: Revert the revert of removal of usless move insns.

2019-04-20 Thread Vladimir Makarov
On 4/20/19 4:55 AM, Uros Bizjak wrote: On 4/20/19, Vladimir Makarov wrote: On 11/21/18 2:33 PM, Uros Bizjak wrote: Hello! Before the recent patch to post-reload mode switching, vzeroupper insertion depended on the existence of the return copy instructions pair in functions that return a

Re: [PATCH 14/14] Add D Phobos config, makefiles, and testsuite.

2019-04-20 Thread Iain Buclaw
On Sat, 20 Apr 2019 at 22:30, Thomas Schwinge wrote: > > Hi! > > On Tue, 18 Sep 2018 02:39:46 +0200, Iain Buclaw > wrote: > > This patch adds the configure and make files used for building D > > runtime and Phobos. As well as running all unittests and the > > testsuite. > > With a

Re: [PATCH 14/14] Add D Phobos config, makefiles, and testsuite.

2019-04-20 Thread Thomas Schwinge
Hi! On Tue, 18 Sep 2018 02:39:46 +0200, Iain Buclaw wrote: > This patch adds the configure and make files used for building D > runtime and Phobos. As well as running all unittests and the > testsuite. With a x86_64-pc-linux-gnu build, I've noticed breakage in '-m32' multilib testing, made

Re: [PATCH] PR fortran/90166 -- check F2018:C1547

2019-04-20 Thread Dominique d'Humières
OK I missed the previous error. However I am still puzzled by (2): + +found outside of a module Dominique > Le 20 avr. 2019 à 21:18, Steve Kargl a > écrit : > > On Sat, Apr 20, 2019 at 09:57:34AM -0700, Steve Kargl wrote: >> On Sat, Apr 20, 2019 at 05:38:34PM +0200, Dominique d'Humières

Re: [PATCH] PR fortran/90166 -- check F2018:C1547

2019-04-20 Thread Steve Kargl
On Sat, Apr 20, 2019 at 09:57:34AM -0700, Steve Kargl wrote: > On Sat, Apr 20, 2019 at 05:38:34PM +0200, Dominique d'Humières wrote: > > > > The changes in gfortran.dg/submodule_22.f08 look weird: > > (1) is the error in the CONTAINS of a SUBMODULE invalid? > > From > > > > * decl.c

New French PO file for 'gcc' (version 9.1-b20190414)

2019-04-20 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 French team of translators. The file is available at: https://translationproject.org/latest/gcc/fr.po (This file, 'gcc-9.1-b20190414.fr.po',

Re: [PATCH] PR fortran/90166 -- check F2018:C1547

2019-04-20 Thread Paul Richard Thomas
It looks good to me, modulo Dominique's query. OK for trunk (note from Richi - still aiming for zero P1 bugs so you're good to go). Thanks Paul On Fri, 19 Apr 2019 at 22:44, Steve Kargl wrote: > > The attached patch fixes PR fortran/91066. The original > code was feeding a nonsense string of

[PATCH, PR d/89293] Committed core.atomic should have fallback when there's no libatomic

2019-04-20 Thread Iain Buclaw
Hi, This patch adds an implementation of core.atomic for when there's no libatomic support linked in, nor provided by the compiler. The main part of which fakes the purity of system mutex_lock/mutex_unlock in order to satisfy the using of it in the pure/nothrow/safe atomic functions. Regression

Re: [PATCH] PR fortran/90166 -- check F2018:C1547

2019-04-20 Thread Steve Kargl
On Sat, Apr 20, 2019 at 05:38:34PM +0200, Dominique d'Humières wrote: > > The changes in gfortran.dg/submodule_22.f08 look weird: > (1) is the error in the CONTAINS of a SUBMODULE invalid? > From > > * decl.c (in_module_or_interface): New function to check that the > current state is

Re: [PATCH] PR fortran/90166 -- check F2018:C1547

2019-04-20 Thread Dominique d'Humières
Hi Steve, The changes in gfortran.dg/submodule_22.f08 look weird: (1) is the error in the CONTAINS of a SUBMODULE invalid? From * decl.c (in_module_or_interface): New function to check that the current state is in a module, submodule, or interface. it should not, should it? (2)

Re: [PATCH, LRA]: Revert the revert of removal of usless move insns.

2019-04-20 Thread Uros Bizjak
On 4/20/19, Vladimir Makarov wrote: > > On 11/21/18 2:33 PM, Uros Bizjak wrote: >> Hello! >> >> Before the recent patch to post-reload mode switching, vzeroupper >> insertion depended on the existence of the return copy instructions >> pair in functions that return a value. The first instruction

Re: [PATCH v3] Use builtin sort instead of shell sort

2019-04-20 Thread Andreas Schwab
On Apr 20 2019, Émeric Dupont wrote: > The following patch fixes this issue by using the [Make $(sort list)][1] > function instead to remove duplicates from the list of headers. There is > no functional change, the value assigned to the shell variable is the > same. Is it? > diff --git

[PATCH v3] Use builtin sort instead of shell sort

2019-04-20 Thread Émeric Dupont
Some build environments and configuration options may lead to the make variable PLUGIN_HEADERS being too long to be passed as parameters to the shell `echo` command, leading to a "write error" message when making the target install-plugin. The following patch fixes this issue by using the [Make

[C++ PATCH] PR c++/90190 - CTAD with list-constructor.

2019-04-20 Thread Jason Merrill
The passage quoted talks about an initializer list containing a single expression, but a braced-init-list is not an expression. Tested x86_64-pc-linux-gnu, applying to trunk. * pt.c (do_class_deduction): Don't try the single element deduction if the single element is also a

[C++ PATCH] PR c++/90171 - ICE with destroying delete with size_t parm.

2019-04-20 Thread Jason Merrill
The problem here was that "second parm is size_t" is false for a destroying sized delete. So let's introduce sized_deallocation_fn_p when that's what we're asking, and reserve second_parm_is_size_t for the specific case of warning about possible confusion with placement delete. Tested