[doc, committed] document that --help and --help= options cannot be combined

2018-11-17 Thread Sandra Loosemore
This patch is for PR 31357. I've applied the patch in the issue with a minor copy-edit. -Sandra 2018-11-17 Nick Clifton Sandra Loosemore PR driver/31357 gcc/ * doc/invoke.texi (Overall Options): Document that --help and --help= options cannot be combined. Index:

OpenACC ICV acc-default-async-var (was: [gomp4] Async related additions to OpenACC runtime library)

2018-11-17 Thread Thomas Schwinge
Hi Chung-Lin! On Mon, 13 Feb 2017 18:13:42 +0800, Chung-Lin Tang wrote: > This patch adds: > > // New functions to set/get the current default async queue > void acc_set_default_async (int); > int acc_get_default_async (void); > > and _async versions of a few existing API functions: (Please,

Re: [PATCH] Implement std::pmr::synchronized_pool_resource

2018-11-17 Thread Jonathan Wakely
On 08/11/18 18:02 +, Jonathan Wakely wrote: Define the thread-safe pool resource, using a shared_mutex to allow multiple threads to concurrently allocate from thread-specific pools. Define new weak symbols for the pthread_rwlock_t functions, to avoid making libstdc++.so depend on

[PATCH] RISC-V: Fix epilogue unwind info with fp and single sp adjust.

2018-11-17 Thread Jim Wilson
Without this patch, compiling a non-leaf function with -g -O0, we get addis0,sp,16 .cfi_def_cfa 8, 0 ... lw s0,8(sp) .cfi_restore 8 addisp,sp,16 .cfi_def_cfa_register 2 which clobbers the frame pointer before resetting the cfa to

[PATCH] avoid error_mark_node in -Wsizeof-pointer-memaccess (PR 88065)

2018-11-17 Thread Martin Sebor
-Wsizeof-pointer-memaccess fails with an ICE when one of the arguments is ill-formed (error_mark_node). To avoid the error the attached patch has the function bail in this case. Martin PR c/88065 - ICE in -Wsizeof-pointer-memaccess on an invalid strncpy gcc/c-family/ChangeLog: PR c/88065 *

Re: [PATCH] Support simd function declarations via a pre-include. (was: [PATCH][RFC]Overloading intrinsics)

2018-11-17 Thread Bernhard Reutner-Fischer
On 15 November 2018 21:54:23 CET, Jakub Jelinek wrote: >> Can we use plain -include like in C? > >Wouldn't that be confusing whether it is included in preprocessor only >or if >it is included as a magic fortran include line at the beginning? Yes, of course. Forgot that its a cpp argument. So

Re: Fix hashtable node deallocation

2018-11-17 Thread François Dumont
Here is the same patch but this time with a test change which is supposed to show the problem. However it doesn't because of:   _Pointer_adapter(element_type* __arg = 0)   { _Storage_policy::set(__arg); } which is not explicit. So is this patch really necessary ? If it isn't, is usage

Re: [PATCH] Fix debuginfo in -fopenmp code (PR debug/87039)

2018-11-17 Thread Jakub Jelinek
On Sat, Nov 17, 2018 at 05:51:05PM +0100, Richard Biener wrote: > On November 17, 2018 4:14:58 PM GMT+01:00, Jakub Jelinek > wrote: > >On Sat, Nov 17, 2018 at 08:31:32AM +0100, Richard Biener wrote: > >> On November 16, 2018 10:10:00 PM GMT+01:00, Jakub Jelinek > >> wrote: Can you add a comment

Re: [PATCH] Support simd function declarations via a pre-include.

2018-11-17 Thread Bernhard Reutner-Fischer
Hi >I'm sending version, I changed the container to hash_map that should >provide >faster look up. > >I've been testing the patch right now. In find_fortran_preinclude_file() you allocate the filename. diff --git a/gcc/fortran/scanner.c b/gcc/fortran/scanner.c index 55d6dafdb5d..4e500f88174

Re: [C++ Patch] Remove obsolete _vptr check (and fix location)

2018-11-17 Thread Paolo Carlini
Hi again... On 17/11/18 12:56, Paolo Carlini wrote: Never mind, got confused for various reasons... too bad anyway that we unconditionally reject such member name even when there is no virtual pointer around. Anyway, I'll send a new, straightforward patch only fixing the two locations. ...

Re: [PATCH] Support simd function declarations via a pre-include.

2018-11-17 Thread Paul Richard Thomas
Hi All, Forget my remark about mp_prop_design.f90. ifort -parallel means just that and has nothing to do with vectorization. Sorry for the noise. Paul On Sat, 17 Nov 2018 at 13:29, Paul Richard Thomas wrote: > > Hi All, > > I took a few moments away from what I really must be doing to try out

[doc, committed] document implicit extern "C" block in system headers

2018-11-17 Thread Sandra Loosemore
This patch is for PR4225, another ancient C++ documentation issue, in which a user expressed surprise at the implicit extern "C" block the preprocessor added to system headers specified via CPLUS_INCLUDE_PATH. Presently RS/6000 AIX is the only remaining target that gives this special

Re: [PATCH] Fix debuginfo in -fopenmp code (PR debug/87039)

2018-11-17 Thread Richard Biener
On November 17, 2018 4:14:58 PM GMT+01:00, Jakub Jelinek wrote: >On Sat, Nov 17, 2018 at 08:31:32AM +0100, Richard Biener wrote: >> On November 16, 2018 10:10:00 PM GMT+01:00, Jakub Jelinek >> wrote: Can you add a comment why doing it >differently >> for this case is necessary or do it the same

Re: *Ping* Fix PR 70260, ICE on invalid

2018-11-17 Thread Paul Richard Thomas
Hi Thomas, OK for trunk. Thanks for working on it. Paul On Sat, 17 Nov 2018 at 15:10, Thomas Koenig wrote: > > Hi, > > > the attached patch fixes both ICEs in the PR by adding some tests. > > It was necessary to shuffle around a bit of code, plus to make sure that > > double error reporting

[doc, committed] clarify -fno-implicit-templates usage

2018-11-17 Thread Sandra Loosemore
This patch is for PR 4025, one of the oldest documentation issues in bugzilla. I think that since 2001 (GCC 3.0) expectations for C++ template handling have evolved so that users expect it to "just work" without messing with options like this, but it's a trivial fix to the docs. -Sandra

Re: [patch, fortran] Fix PR 70260, ICE on invalid

2018-11-17 Thread Jerry DeLisle
On 11/11/18 7:59 AM, Thomas Koenig wrote: Hello world, the attached patch fixes both ICEs in the PR by adding some tests. It was necessary to shuffle around a bit of code, plus to make sure that double error reporting did not become too bad. Regression-tested. OK for trunk? Regards

[PATCH, libphobos] Committed IEEE quadruple support to std.conv

2018-11-17 Thread Iain Buclaw
This patch is backported from phobos 2.079, in continuation of finishing off AArch64 library support. Bootstrapped and ran D2 testsuite on x86_64-linux-gnu. Committed to trunk as r266238. -- Iain --- diff --git a/libphobos/src/std/conv.d b/libphobos/src/std/conv.d index

Re: [PATCH] Fix debuginfo in -fopenmp code (PR debug/87039)

2018-11-17 Thread Jakub Jelinek
On Sat, Nov 17, 2018 at 08:31:32AM +0100, Richard Biener wrote: > On November 16, 2018 10:10:00 PM GMT+01:00, Jakub Jelinek > wrote: Can you add a comment why doing it differently > for this case is necessary or do it the same way in all cases? Do you mean in omp-expand.c or dwarf2out.c? I

*Ping* Fix PR 70260, ICE on invalid

2018-11-17 Thread Thomas Koenig
Hi, the attached patch fixes both ICEs in the PR by adding some tests. It was necessary to shuffle around a bit of code, plus to make sure that double error reporting did not become too bad. Regression-tested. OK for trunk? Ping? Regards Thomas

[PATCH, libphobos] Fix libphobos.shared testsuite for multilib tests

2018-11-17 Thread Johannes Pfau
Hi, the loadDR test in the libphobos.shared testsuite tries to dynamically load the phobos library. The path for the library currently points to the main multilib variant phobos library, causing other multilib variants to fail the test. The attached patch uses $blddir instead of $objdir to fix

Re: [PATCH 0/2] asm qualifiers (PR55681) and asm input

2018-11-17 Thread Segher Boessenkool
Ping x2. On Sat, Nov 10, 2018 at 06:33:37PM -0600, Segher Boessenkool wrote: > Ping. > > On Tue, Oct 30, 2018 at 05:30:32PM +, Segher Boessenkool wrote: > > Hi! > > > > This is the same "asm input" patch as before, but now preceded by a patch > > that makes all orderings of

Re: [PATCH 21/25] GCN Back-end (part 2/2).

2018-11-17 Thread Segher Boessenkool
On Fri, Nov 16, 2018 at 10:09:59AM -0600, Segher Boessenkool wrote: > On Mon, Nov 12, 2018 at 11:54:58AM -0700, Jeff Law wrote: > > On 11/12/18 10:52 AM, Andrew Stubbs wrote: > > > If there are two instructions that both have an UNSPEC_VOLATILE, will > > > combine coalesce them into one in the

Re: [PATCH] Support simd function declarations via a pre-include.

2018-11-17 Thread Paul Richard Thomas
Hi All, I took a few moments away from what I really must be doing to try out an earlier version of the patch. There are quite a few CRs in the patch and the third chunk in gcc.c was rejected, although I cannot see why. I made a change to scanner.c to prevent the segfault that results from not

Re: [C++ Patch] Remove obsolete _vptr check (and fix location)

2018-11-17 Thread Paolo Carlini
Hi, Il 17 Novembre 2018 11:00:45 CET, Paolo Carlini ha scritto: >Hi, > >while I was working on some location issues I noticed this check which >seems obsolete to me and means that we unnecessarily reject kosher >code: >indeed we don't test for it anywhere and neither ICC nor clang enforce

Fix ICE while outputting ODR warnings

2018-11-17 Thread Jan Hubicka
Hi, this patch fixes ICE with the testcase in PR 87957 which is caused by type variants now having only IDENTIFIER_NODE as TYPE_NAME. I did not add the testcase yet becasue bit more fixing is needed. The code outputs incorrectly warning about subtype (it misses there is ODR violation in it and

[PATCH, PR d/87824] Committed return min_align_of_type from alignsize hook.

2018-11-17 Thread Iain Buclaw
Hi, This patch addresses some of the failing tests on x86_64-linux/-m32. The failing tests in question checked that data with 'long' was sufficiently aligned. assert(( % long.alignof) == 0); Where long.alignof was replaced with the result of TYPE_ALIGN_UNIT. The D language expects the

[C++ Patch] Remove obsolete _vptr check (and fix location)

2018-11-17 Thread Paolo Carlini
Hi, while I was working on some location issues I noticed this check which seems obsolete to me and means that we unnecessarily reject kosher code: indeed we don't test for it anywhere and neither ICC nor clang enforce it. I also went through the SVN history and the check is *extremely* old,