[PATCH] libgomp: Add OMPD process functions and datatypes.

2020-07-03 Thread y2s1982 via Gcc-patches
This patch adds OMPD functions defined in 5.5.2 of the OpenMP 5.0 API documentation. It adds per-process and per-device functions, defines related handle data types, and adds a helper function for storing device id. 2020-07-03 Tony Sim libgomp/ChangeLog: * Makefile.am (libgompd_la_OBJ

Re: [PATCH] c++: Make convert_like complain about bad ck_ref_bind again [PR95789]

2020-07-03 Thread Jason Merrill via Gcc-patches
On 6/22/20 10:09 PM, Marek Polacek wrote: convert_like issues errors about bad_p conversions at the beginning of the function, but in the ck_ref_bind case, it only issues them after we've called convert_like on the next conversion. This doesn't work as expected since r10-7096 because when we see

Re: [PATCH] Add TARGET_LOWER_LOCAL_DECL_ALIGNMENT [PR95237]

2020-07-03 Thread Jason Merrill via Gcc-patches
On 6/29/20 5:00 AM, Richard Biener wrote: On Fri, Jun 26, 2020 at 10:11 PM H.J. Lu wrote: On Thu, Jun 25, 2020 at 1:10 AM Richard Biener wrote: On Thu, Jun 25, 2020 at 2:53 AM Sunil Pandey wrote: On Wed, Jun 24, 2020 at 12:30 AM Richard Biener wrote: On Tue, Jun 23, 2020 at 5:31 PM Su

Re: [PATCH] avoid -Wnonnull for lambda stubs (PR c++/95984)

2020-07-03 Thread Jason Merrill via Gcc-patches
On 7/2/20 11:21 AM, Martin Sebor wrote: On 7/1/20 3:25 PM, Jason Merrill wrote: On 7/1/20 3:31 PM, Martin Sebor wrote: The attached patch avoids null pointer checking for the first argument to calls to the member operator() of lambda objects emitted by the C++ front end.  This avoids both the s

[PATCH] PR fortran/95709 - [9/10/11 Regression] ICE in gfc_resolve_code, at fortran/resolve.c:11807

2020-07-03 Thread Harald Anlauf
> The obsolescent (=legacy) assigned GOTO should only allow scalar integer > variables. Check for proper conditions. I've played around some more with invalid uses of assigned GOTO and found that some cases produced a suboptimal error message. The attached patch (v2) improves upon this and provi

Re: [PATCH] libgomp: Add OMPD per-process functions.

2020-07-03 Thread y2s1982 . via Gcc-patches
Hello, I believe I misused the write_memory() function. This function is currently used in ompd_device_initialize() to write a block of memory that represents the id of the device, stored in the handle as a pointer. I would need to resubmit the patch but I have some questions regarding the issue.

[wwwdocs] Document new G++ features

2020-07-03 Thread Marek Polacek via Gcc-patches
Pushed. commit c325a1addc0fc612a9db218be6abb41b2052a447 Author: Marek Polacek Date: Fri Jul 3 14:14:34 2020 -0400 gcc-11/changes:: Document new C++ stuff. diff --git a/htdocs/gcc-11/changes.html b/htdocs/gcc-11/changes.html index cea01a9c..287dc864 100644 --- a/htdocs/gcc-11/changes.html

[wwwdocs] Another update of the C++ DR table

2020-07-03 Thread Marek Polacek via Gcc-patches
Pushed. commit 265e9e1ea8117f34f6f97b7eb7be6df119f2c646 Author: Marek Polacek Date: Wed Jul 1 11:07:21 2020 -0400 More updates to the C++ DRs table. diff --git a/htdocs/projects/cxx-dr-status.html b/htdocs/projects/cxx-dr-status.html index 482b59b0..92d72121 100644 --- a/htdocs/projects/

[PATCH] libgomp: Add OMPD per-process functions.

2020-07-03 Thread y2s1982 via Gcc-patches
This patch adds OMPD process and device handling functions and related data structures as described in the OMP 5.0 API documentation, section 5.5.2. 2020-07-03 Tony Sim libgomp/ChangeLog: * Makefile.am (libgompd_la_SOURCES): Add ompd-proc.c * Makefile.in: Regenerate. *

[PATCH] nvptx: : Add support for popcount and widening multiply instructions

2020-07-03 Thread Roger Sayle
The following patch adds support for three-input addition instructions to the nvptx backend. The PTX ISA's "vadd.u32.u32.u32.add d, a, b, c" instruction effectively implements 32-bit d = a+b+c, and the "vsub.u32.u32.u32 d,a,b,c" instruction that provides 32-bit d = (a-b)+c. The hope is that t

Re: [PATCH] Add -fld-path= to specify an arbitrary executable as the linker

2020-07-03 Thread Fāng-ruì Sòng via Gcc-patches
On 2020-07-03, Martin Liška wrote: On 7/2/20 9:34 PM, Fāng-ruì Sòng wrote: On 2020-07-01, Fāng-ruì Sòng wrote: On 2020-07-01, Martin Liška wrote: On 6/30/20 5:32 PM, Fāng-ruì Sòng wrote: There is some concern about clang's -fuse-ld=path http://lists.llvm.org/pipermail/cfe-dev/2020-June/0657

Re: [PATCH 2/3] Don't copy back vars mapped with acc_map_data

2020-07-03 Thread Thomas Schwinge
Hi! On 2020-01-17T13:18:20-0800, Julian Brown wrote: > This patch prevents "exit data" directives from copying back data that > was mapped with an acc_map_data API call. This matches the behaviour > expected by the pr92843-1.c test > --- a/libgomp/oacc-mem.c > +++ b/libgomp/oacc-mem.c > @@ -1235

Re: [PATCH 3/9] [OpenACC] Adjust dynamic reference count semantics

2020-07-03 Thread Thomas Schwinge
Hi Julian! On 2020-06-30T15:51:14+0200, I wrote: > On 2020-06-16T15:38:33-0700, Julian Brown wrote: >> This is a new version of the patch last sent here: >> >> https://gcc.gnu.org/pipermail/gcc-patches/2020-May/546332.html >> >> Minus the bits that Thomas has committed already (thanks!), and with

[PATCH] ipa-sra: Avoid transitive splits with type mismatches (PR 96040)

2020-07-03 Thread Martin Jambor
Hi, PR 96040 revealed IPA-SRA, when checking whether an intended split is the same as the one in a called function does not also check if the types match and the transformation code does not handle any resulting type mismatches. This patch simply avoids the the split in the case of mismatches, so

Re: [PATCH 02/13] OpenACC reference count overhaul

2020-07-03 Thread Thomas Schwinge
Hi! To move us one small step forward: On 2020-06-25T13:03:53+0200, I wrote: > Ping, in particular my question about different 'GOMP_MAP_FORCE_FROM' vs. > 'GOMP_MAP_FROM' handling. > > (I have not yet looked whether 'GOMP_MAP_ALWAYS_FROM' may be generate > nowadays, given your pending front end/m

Re: [Patch 2/3] aarch64: Introduce SLS mitigation for RET and BR instructions

2020-07-03 Thread Matthew Malcomson
With suggestions applied. Testing with `-mabi=ilp32` found a bug around the trampoline initialisation where the new larger size of the trampoline caused a different execution path of `emit_block_move` which ICE'd on the pre-existing `ptr_mode` address. Commit Message -- Instructions followin

Re: [Patch 1/3] aarch64: New Straight Line Speculation (SLS) mitigation flags

2020-07-03 Thread Matthew Malcomson
With suggestions applied. Here we introduce the flags that will be used for straight line speculation. The new flag introduced is `-mharden-sls=`. This flag can take arguments of `none`, `all`, or a comma seperated list of one or more of `retbr` or `blr`. `none` indicates no special mitigation o

Re: Fortran : Fortran translation issues PR52279

2020-07-03 Thread David Edelsohn via Gcc-patches
Mark, A full bootstrap is successful with the translation markers restored and declaring hint as "const char *". const char *hint = _(" [see %<-fno-allow-invalid-boz%>]"); I assume that the translation system works correctly for that style. Do you want to apply the patch or do you want me to?

Re: [PATCH] Add -fld-path= to specify an arbitrary executable as the linker

2020-07-03 Thread Martin Liška
On 7/2/20 9:34 PM, Fāng-ruì Sòng wrote: On 2020-07-01, Fāng-ruì Sòng wrote: On 2020-07-01, Martin Liška wrote: On 6/30/20 5:32 PM, Fāng-ruì Sòng wrote: There is some concern about clang's -fuse-ld=path http://lists.llvm.org/pipermail/cfe-dev/2020-June/065710.html and use of COMPILER_PATH vs PA

Re: [PATCH][RFC] Do not stream all zeros for gcda files.

2020-07-03 Thread Gerald Pfeifer
On Fri, 3 Jul 2020, Rainer Orth wrote: > I'm seeing the same on both i386-pc-solaris2.11 and > sparc-sun-solaris2.11. It's in stage2, so the bootstrap compiler > (gcc 8 in my case) should be immaterial. Yes, that's what I meant (but did not articulate well). > The following patch allowed bootst

Re: [PATCH] gcov: fix gcov-tool merge for TOPN counters

2020-07-03 Thread Martin Liška
On 7/2/20 9:03 PM, Gerald Pfeifer wrote: On Tue, 16 Jun 2020, Martin Liška wrote: libgcc/ChangeLog: * libgcov-util.c (read_gcda_finalize): Remove const operator. (merge_wrapper): Add both counts and use them properly. (topn_to_memory_representation): New function.

Re: [PATCH][RFC] Do not stream all zeros for gcda files.

2020-07-03 Thread Martin Liška
Hi. I'm going to install the following, tested with -m32. Martin >From 6c9e35a569f5a46fed7c8de6ac22545cb845a913 Mon Sep 17 00:00:00 2001 From: Martin Liska Date: Fri, 3 Jul 2020 13:45:45 +0200 Subject: [PATCH] gcov-dump: fix build for i386 gcc/ChangeLog: PR bootstrap/96046 * gcov-dump.c (ta

Re: [PATCH][RFC] Do not stream all zeros for gcda files.

2020-07-03 Thread Martin Liška
On 7/3/20 10:46 AM, Rainer Orth wrote: Hi Gerald, On Thu, 2 Jul 2020, Martin Liška wrote: All right, you convinced me and I'm going to install the patch. I'm fraid this may have broke i386-unknown-freebsd-11.4 (with clang 10.0 as bootstrap compiler, though that doesn't appear to be the trigg

Re: [patch] Extend store merging to STRING_CST

2020-07-03 Thread Richard Biener via Gcc-patches
On Thu, Jul 2, 2020 at 9:08 PM Eric Botcazou wrote: > > > So this variant combined with the rest of the patch is OK then. > > Thanks. It occurred to me that using string_constant might be slightly better > (iti is already used by gimple_fold_builtin_memchr in the same file). OK. Thanks, Richard

Re: [PATCH] OpenMP: Disable GPU threads when only teams are used

2020-07-03 Thread Jakub Jelinek via Gcc-patches
On Thu, Jul 02, 2020 at 10:16:25PM +0100, Andrew Stubbs wrote: > On 02/07/2020 18:00, Jakub Jelinek wrote: > > On Thu, Jul 02, 2020 at 05:15:20PM +0100, Andrew Stubbs wrote: > > > This patch, originally by Kwok, auto-adjusts the default OpenMP target > > > arguments to set num_threads(1) when there

[PATCH] tree-optimization/96037 - fix uninitialized use of slp_op

2020-07-03 Thread Richard Biener
The following avoids leaving slp_def as passed to vect_is_simple_use by reference uninitialized. Bootstrapped and tested on x86_64-unknown-linux-gnu, pushed. 2020-07-03 Richard Biener PR tree-optimization/96037 * tree-vect-stmts.c (vect_is_simple_use): Initialize *slp_def. ---

Re: [Patch][gcn, nvptx, offloading] mkoffload – handle -fpic/-fPIC

2020-07-03 Thread Tom de Vries
On 6/26/20 9:35 PM, Moore, Catherine wrote: > Hi Tom, > > It doesn't look like you were explicitly cc'd on this patch and probably > haven't seen it. Would you mind taking a look and approving the nvptx > portions? > [ thanks for the ping, I think was explicitly cc-ed though. I'm usually not

[committed][OG10] amdgcn: Add fold_left_plus vector reductions

2020-07-03 Thread Andrew Stubbs
Now backported to OG10. Andrew On 03/07/2020 11:11, Andrew Stubbs wrote: This patch implements a floating-point fold_left_plus vector pattern, which gives a significant speed-up in the BabelStream "dot" benchmark. The GCN architecture can't actually do an in-order vector reduction any more e

[committed] amdgcn: Add fold_left_plus vector reductions

2020-07-03 Thread Andrew Stubbs
This patch implements a floating-point fold_left_plus vector pattern, which gives a significant speed-up in the BabelStream "dot" benchmark. The GCN architecture can't actually do an in-order vector reduction any more efficiently than that equivalent scalar algorithm, so this is a bit of a che

[PATCH] fix scalar BB vectorization costing

2020-07-03 Thread Richard Biener
We were costing the scalar pattern stmts rather than the scalar original stmt and also not appropriately looking at the pattern stmt for whether the stmt is vectorized. Bootstrap and regtest running on x86_64-unknown-linux-gnu. 2020-07-03 Richard Biener * tree-vect-slp.c (vect_bb_slp_

Re: [PATCH][RFC] Do not stream all zeros for gcda files.

2020-07-03 Thread Rainer Orth
Hi Gerald, > On Thu, 2 Jul 2020, Martin Liška wrote: >> All right, you convinced me and I'm going to install the patch. > > I'm fraid this may have broke i386-unknown-freebsd-11.4 (with clang 10.0 > as bootstrap compiler, though that doesn't appear to be the trigger here): > > /scratch/tmp/gerald

[PATCH] refactor SLP constant insertion and provde entry insert helper

2020-07-03 Thread Richard Biener
This provides helpers to insert stmts on region entry abstracted from loop/basic-block split out from vec_init_vector and used from the SLP constant code generation path. The SLP constant code generation path is also changed to avoid needless SSA copying since we can store VECTOR_CSTs directly in