Re: [PATCH 2/7] PowerPC tests: Add PLI/PADDI tests.

On Thu, Jun 25, 2020 at 11:52:50AM -0500, Segher Boessenkool wrote: > On Mon, Jun 01, 2020 at 03:53:37PM -0400, Michael Meissner wrote: > > Add tests for -mcpu=future that test the generation of PADDI (and PLI which > > becomes PADDI). > > > > 2020-06-01 Michael Meissner > > > > *

Re: [PATCH 5/7, V2] PowerPC tests: Prefixed insn with large offsets

On Thu, Jun 25, 2020 at 12:09:41PM -0500, Segher Boessenkool wrote: > Hi! > > On Thu, Jun 04, 2020 at 01:03:51PM -0400, Michael Meissner wrote: > > [PATCH 5/7, V2] PowerPC tests: Prefixed insn with large offsets > > > > Add tests to make sure for -mcpu=future that prefixed load/store > >

Re: [PATCH 7/7] PowerPC test: Add prefixed stack protect test

On Thu, Jun 25, 2020 at 12:18:42PM -0500, Segher Boessenkool wrote: > Hi! > > On Mon, Jun 01, 2020 at 03:53:42PM -0400, Michael Meissner wrote: > > Test that stack protection generates prefixed stack instructions if you are > > using large stack frame for -mcpu=future. > > > > 2020-06-01

Re: [PATCH 2/7] PowerPC tests: Add PLI/PADDI tests.

On Thu, Jun 25, 2020 at 11:52:50AM -0500, Segher Boessenkool wrote: > On Mon, Jun 01, 2020 at 03:53:37PM -0400, Michael Meissner wrote: > > Add tests for -mcpu=future that test the generation of PADDI (and PLI which > > becomes PADDI). > > > > 2020-06-01 Michael Meissner > > > > *

Re: [PATCH] Fix target clone indirection elimination.

On Fri, Jun 26, 2020 at 5:51 PM Jeff Law wrote: > > On Mon, 2020-06-22 at 11:24 +0200, Richard Biener via Gcc-patches wrote: > > On Sun, Jun 21, 2020 at 2:32 AM Yichao Yu via Gcc-patches > > wrote: > > > On Sat, Jun 20, 2020 at 8:16 PM Yichao Yu wrote: > > > > On Sat, Jun 20, 2020 at 3:41 PM

Re: [PATCH] tree-ssa-threadbackward.c (profitable_jump_thread_path): Do not allow __builtin_constant_p.

On Fri, 2020-06-26 at 16:04 -0600, Jeff Law wrote: > On Fri, 2020-06-26 at 23:54 +0200, Ilya Leoshkevich wrote: > > How should this work ideally? Suppose we have: > > > > static inline void foo (int i) > > { > > if (__builtin_is_constant_p (i)) > > asm volatile ("bar" :: "i" (i)) > >

Re: [PATCH] arc: add exceptions for PR92860.

On Wed, 2020-06-24 at 09:43 +0200, Martin Liška wrote: > Hey. > > The patch is about addition of some exceptions for arc target that > address: > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=92860#c26 > > It's again another example where optimization options influence target > options. > >

Re: [PATCH, rs6000] Add support to enable vmsumudm behind vec_msum builtin.

Hi! On Fri, Jun 26, 2020 at 01:20:43PM -0500, will schmidt wrote: > * config/rs6000/altivec.h (vec_vmsumudm): New define. > * config/rs6000/altivec.md (UNSPEC_VMSUMUDM): New unspec. > (altivec_vmsumudm): New define_insn. > * config/rs6000/rs6000-builtin.def

Re: [PATCH] Linux/i386: Remove SUBTARGET_FRAME_POINTER_REQUIRED

On Fri, 2020-06-12 at 05:56 -0700, H.J. Lu wrote: > On Thu, Jun 11, 2020 at 8:24 PM Jeff Law wrote: > > On Sat, 2020-05-02 at 04:55 -0700, H.J. Lu wrote: > > > Currently patchable area is at the wrong place. It is placed immediately > > > after function label, before both .cfi_startproc and

Re: [PATCH] tree-ssa-threadbackward.c (profitable_jump_thread_path): Do not allow __builtin_constant_p.

On Fri, 2020-06-26 at 23:54 +0200, Ilya Leoshkevich wrote: > > How should this work ideally? Suppose we have: > > static inline void foo (int i) > { > if (__builtin_is_constant_p (i)) > asm volatile ("bar" :: "i" (i)) > else > asm volatile ("baz" :: "d" (i)); > } > > First of all,

Re: [PING][PATCH] underline null argument in -Wnonnull (PR c++/86568)

On Mon, 2020-06-22 at 12:02 -0600, Martin Sebor wrote: > Ping: https://gcc.gnu.org/pipermail/gcc-patches/2020-June/547415.html > > Jason already approved the C++ changes (with a couple of minor > tweaks). I'm still looking for an approval of the corresponding > middle end diff. Do you need an

Re: [PATCH] tree-ssa-threadbackward.c (profitable_jump_thread_path): Do not allow __builtin_constant_p.

On Fri, 2020-06-26 at 15:34 -0600, Jeff Law wrote: > On Fri, 2020-06-26 at 23:17 +0200, Ilya Leoshkevich wrote: > > On Fri, 2020-06-26 at 14:27 -0600, Jeff Law wrote: > > > On Thu, 2020-06-04 at 02:37 +0200, Ilya Leoshkevich via Gcc- > > > patches > > > wrote: > > > > Bootstrapped and regtested on

Re: [PATCH] Fix target clone indirection elimination.

On Mon, 2020-06-22 at 11:24 +0200, Richard Biener via Gcc-patches wrote: > On Sun, Jun 21, 2020 at 2:32 AM Yichao Yu via Gcc-patches > wrote: > > On Sat, Jun 20, 2020 at 8:16 PM Yichao Yu wrote: > > > On Sat, Jun 20, 2020 at 3:41 PM Yichao Yu wrote: > > > > On Sat, Jun 20, 2020 at 3:26 PM

Re: [PATCH v3] c++: Fix CTAD for aggregates in template [PR95568]

On Fri, Jun 26, 2020 at 04:19:08PM -0400, Jason Merrill wrote: > Please also test scoped enum. Here: -- >8 -- 95568 complains that CTAD for aggregates doesn't work within requires-clause and it turned out that it doesn't work when we try the deduction in a template. The reason is that

Re: [PATCH] tree-ssa-threadbackward.c (profitable_jump_thread_path): Do not allow __builtin_constant_p.

On Fri, 2020-06-26 at 23:17 +0200, Ilya Leoshkevich wrote: > On Fri, 2020-06-26 at 14:27 -0600, Jeff Law wrote: > > On Thu, 2020-06-04 at 02:37 +0200, Ilya Leoshkevich via Gcc-patches > > wrote: > > > Bootstrapped and regtested on x86_64-redhat-linux, ppc64le-redhat- > > > linux > > > and

Re: Move simplification of statements using ranges into its own class.

On Thu, 2020-06-18 at 13:24 +0200, Aldy Hernandez wrote: > Howdy. > > This moves all the simplification code from vr_values into a separate > class (simplify_using_ranges). In doing so, we get rid of a bunch of > dependencies on the internals of vr_values. The goal is to (a) remove >

Re: [PATCH] tree-ssa-threadbackward.c (profitable_jump_thread_path): Do not allow __builtin_constant_p.

On Fri, 2020-06-26 at 14:27 -0600, Jeff Law wrote: > On Thu, 2020-06-04 at 02:37 +0200, Ilya Leoshkevich via Gcc-patches > wrote: > > Bootstrapped and regtested on x86_64-redhat-linux, ppc64le-redhat- > > linux > > and s390x-redhat-linux. > > > > > > Linux Kernel (specifically,

[PATCH] x86-64: Define ASM_OUTPUT_ALIGNED_DECL_LOCAL

Define ASM_OUTPUT_ALIGNED_DECL_LOCAL for large local common symbol. gcc/ChangeLog: * config/i386/x86-64.h (ASM_OUTPUT_ALIGNED_DECL_LOCAL): New. gcc/testsuite/ChangeLog: * gcc.target/i386/pr95620.c: New test. --- gcc/config/i386/x86-64.h| 11 +++

Re: [PATCH] tree-ssa-threadbackward.c (profitable_jump_thread_path): Do not allow __builtin_constant_p.

On Thu, 2020-06-04 at 02:37 +0200, Ilya Leoshkevich via Gcc-patches wrote: > Bootstrapped and regtested on x86_64-redhat-linux, ppc64le-redhat-linux > and s390x-redhat-linux. > > > Linux Kernel (specifically, drivers/leds/trigger/ledtrig-cpu.c) build > with GCC 10 fails on s390 with "impossible

Re: [PATCH v2] c++: Fix CTAD for aggregates in template [PR95568]

On 6/25/20 5:37 PM, Marek Polacek wrote: On Wed, Jun 24, 2020 at 03:52:12PM -0400, Jason Merrill via Gcc-patches wrote: On 6/23/20 6:58 PM, Marek Polacek wrote: 95568 complains that CTAD for aggregates doesn't work within requires-clause and it turned out that it doesn't work when we try the

[PATCH] Add TARGET_LOWER_LOCAL_DECL_ALIGNMENT [PR95237]

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 Sunil K Pandey via Gcc-patches > > > wrote: > > > > > > > > From: Sunil K

[RFC] rs6000: Add builtins for fegetround, feclearexcept and feraiseexcept [PR94193]

Hi all, This is an early draft I'm working on to add fegetround , feclearexcept and feraiseexcept as builtins on rs6000. This is my first patch so I welcome any and all feedback. Foremost I have some questions to ask as I got stuck on some problems. Q1) How to implement a target specific

Re: PSA: Default C++ dialect is now C++17

On Fri, Jun 26, 2020 at 03:34:09PM -0400, Marek Polacek via Gcc-patches wrote: > As discussed last month: > > it's time to change the C++ default to gnu++17. I've committed the patch > after > testing x86_64-pc-linux-gnu and

[PATCH] PR fortran/95881 - [9/10/11 Regression] ICE in resolve_symbol, at fortran/resolve.c:15175

Dear all, here's another NULL pointer dereference on invalid code. Regtested on x86_64-pc-linux-gnu. OK for master / backports where appropriate? Thanks, Harald PR fortran/95881 - ICE in resolve_symbol, at fortran/resolve.c:15175 Avoid NULL pointer dereference. gcc/fortran/ PR

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

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, Catherine >-Original Message- >From: Gcc-patches [mailto:gcc-patches-boun...@gcc.gnu.org] On Behalf >Of Burnus,

PSA: Default C++ dialect is now C++17

As discussed last month: it's time to change the C++ default to gnu++17. I've committed the patch after testing x86_64-pc-linux-gnu and powerpc64le-unknown-linux-gnu. Brace yourselves! Marek

[PATCH] c++: Check uniqueness of concepts/variable templates [PR94553]

This patch wraps up PR94553. Variable template names have no C compatibility implications so they should be unique in their declarative region. It occurred to me that this applies to concepts as well. This is not specified in [basic.scope.declarative]/4.2 but that seems like a bug in the

[PATCH, rs6000] Add support to enable vmsumudm behind vec_msum builtin.

Hi, Add support for the vmsumudm instruction and tie it into the vec_msum built-in to support the variants of that built-in using vector _int128 parameters. vector _uint128_t vec_msum (vector unsigned long long, vector unsigned long long,

Re: [PATCH] libstdc++: std::variant doesn't like types with a defaulted virtual destructor [PR95915]

On 26/06/20 19:12 +0300, Ville Voutilainen via Libstdc++ wrote: This patch also deprecates std::is_literal_type. 2020-06-26 Ville Voutilainen PR libstdc++/95915 * include/std/type_traits (is_literal_type, is_literal_type_v): Deprecate in C++17. * include/std/variant

Re: [PATCH] libgomp, fortran: Apply if clause to all sub-constructs in combined OpenMP constructs

On Fri, Jun 26, 2020 at 06:55:30PM +0100, Kwok Cheung Yeung wrote: > Okay for master/OG10? Ok for master, not sure if I have anything to say for OG10. With a nit. > commit 04bdcaa20827d814c323847630c59ee843c51408 > Author: Kwok Cheung Yeung > Date: Fri Jun 26 10:35:36 2020 -0700 > > Fix

Re: [PATCH] libgomp, fortran: Apply if clause to all sub-constructs in combined OpenMP constructs

On 26/06/2020 8:35 am, Christophe Lyon wrote: Hi, I've noticed a regression since your commit, on arm aarch64 and x86: for instance on arm-linux-gnueabi: PASS: gfortran.dg/gomp/combined-if.f90 -O (test for excess errors) PASS: gfortran.dg/gomp/combined-if.f90 -O scan-tree-dump-times

Re: [PATCH] libgomp: added simple functions and tests for OMPD

Hello, On Fri, Jun 26, 2020 at 1:08 PM Jakub Jelinek wrote: > On Fri, Jun 26, 2020 at 12:57:59PM -0400, y2s1982 . wrote: > > > > * env.c(ompd_dll_locations_valid): Define with no compiler > > > optimization. > > > > > > Again, missing space. > > > > > > The contrib/mklog.py, which I use

Re: PING Re: testsuite: clarify scan-dump file globbing behavior

On Jun 2, 2020, at 10:37 PM, Frederik Harwath wrote: > > Frederik Harwath writes: > > ping :-) Ok. >> Frederik Harwath writes: >> >> Hi Rainer, hi Mike, >> ping: https://gcc.gnu.org/pipermail/gcc-patches/2020-May/545803.html >> >> Best regards, >> Frederik >> >>> Hi Thomas, >>> >>>

Re: [PATCH] libgomp: added simple functions and tests for OMPD

On Fri, Jun 26, 2020 at 12:57:59PM -0400, y2s1982 . wrote: > > > * env.c(ompd_dll_locations_valid): Define with no compiler > > optimization. > > > > Again, missing space. > > > > The contrib/mklog.py, which I use to generate these messages, seems to > generate the statement without a space

Re: [PATCH] libgomp: added simple functions and tests for OMPD

Hello, On Fri, Jun 26, 2020 at 5:03 AM Jakub Jelinek wrote: > Thanks, just nits, no need to repost, just commit it after you make those > changes. > > On Thu, Jun 25, 2020 at 05:58:23PM -0400, y2s1982 via Gcc-patches wrote: > > > * Makefile.am(toolexeclib_LTLIBRARIES): Add libgompd.la. >

Re: [PATCH] libstdc++: std::variant doesn't like types with a defaulted virtual destructor [PR95915]

On Fri, 26 Jun 2020 at 19:12, Ville Voutilainen wrote: > > This patch also deprecates std::is_literal_type. I forgot to ask, OK for trunk and GCC 10 if full suite testing passes? The problematic compiler bug has been gone since GCC 10, so we can just as well backport this there, but not further.

[PATCH] libstdc++: std::variant doesn't like types with a defaulted virtual destructor [PR95915]

This patch also deprecates std::is_literal_type. 2020-06-26 Ville Voutilainen PR libstdc++/95915 * include/std/type_traits (is_literal_type, is_literal_type_v): Deprecate in C++17. * include/std/variant (_Uninitialized): Adjust the condition and the comment. *

Re: [PATCH 3/4] libgcc: fix the handling of return address mangling [PR94891]

The 06/05/2020 17:51, Szabolcs Nagy wrote: > Mangling, currently only used on AArch64 for return address signing, > is an internal representation that should not be exposed via > > __builtin_return_address return value, > __builtin_eh_return handler argument, > _Unwind_DebugHook handler

Re: [PATCH 2/4] aarch64: fix __builtin_eh_return with pac-ret [PR94891]

The 06/05/2020 17:51, Szabolcs Nagy wrote: > The handler argument must not be signed since that may come from > outside the current module and exposing signed addresses is a pointer > ABI break. (The signed address also may not be representable as void * > which is why pac-ret is currently broken

Re: [PATCH 1/4] aarch64: fix return address access with pac [PR94891][PR94791]

The 06/05/2020 17:51, Szabolcs Nagy wrote: > This is a big hammer fix for __builtin_return_address (PR target/94891) > returning signed addresses (sometimes, depending on wether lr happens > to be signed or not at the time of call which depends on optimizations), > and similarly -pg may pass

Re: [committed] d: Merge upstream dmd 90450f3ef.

Hi, This patch has been backported, bootstrapped and regression tested on the releases/gcc-9 and releases/gcc-10 branches. Regards Iain. On 25/06/2020 17:41, Iain Buclaw wrote: > Hi, > > This patch merges the D front-end implementation with upstream dmd > 90450f3ef. Fixes a regression caused

Re: [committed] d: Fix ICE in uda_attribute_p when looking up unknown attribute

Hi, This patch has been backported, bootstrapped and regression tested on the releases/gcc-9 and releases/gcc-10 branches. Regards Iain. On 25/06/2020 17:38, Iain Buclaw wrote: > Hi, > > This patch fixes an ICE in uda_attribute_p when looking up an unknown > attribute. The target attribute

Re: [PATCH] rs6000: Add support for __builtin_cpu_is ("power10")

On Fri, Jun 26, 2020 at 03:52:33AM -0400, Michael Meissner wrote: > On Thu, Jun 25, 2020 at 06:54:26PM -0500, Segher Boessenkool wrote: > > On Thu, Jun 25, 2020 at 06:36:51PM -0500, Peter Bergner wrote: > > > * config/rs6000/rs6000-call.c (cpu_is_info) : New. > > > : Remove unneeded ','. > >

Re: [PATCH] coroutines: Handle awaiters that are sub-objects [PR95736]

On 6/26/20 9:02 AM, Iain Sandoe wrote: Hi During development, it seemed a reasonable strategy to defer decisions on what needed to be captured in the coroutine frame (to minimize the frame size). In the case of awaitables that are user-variables, params (or otherwise originate outside the

[PATCH] coroutines: Handle awaiters that are sub-objects [PR95736]

Hi During development, it seemed a reasonable strategy to defer decisions on what needed to be captured in the coroutine frame (to minimize the frame size). In the case of awaitables that are user-variables, params (or otherwise originate outside the coroutine scope), that turns out to make

[pushed] coroutines: Handle bad g-r-o-o-a-f cases.

Hi This fixes an ice-on-invalid I found while working on other cases. tested on x86_64-linux, darwin, powerpc64-linux applied to master as obvious/trivial. thanks Iain --- If we see a get_return_object_on_allocation_failure in the promise, we expect to be able to use it. If this isn't

[PATCH] tree-optimization/95839 - teach SLP vectorization about vector inputs

(sorry for the duplicate, forgot to copy the list) This teaches SLP analysis about vector typed externals that are fed into the SLP operations via lane extracting BIT_FIELD_REFs. It shows that there's currently no good representation for vector code on the SLP side so I went a half way and

Re: [patch] Take into account range info to optimize range tests

On Thu, Jun 25, 2020 at 1:07 PM Eric Botcazou wrote: > > Hi, > > ...into bit tests, as done by optimize_range_tests_to_bit_test of the reassoc > pass. The patch is aimed at addressing the following two issues: > > 1. In order to protect the shift operation from undefinedness, the new bit >

RE: [PATCH] [arm] Don't generate invalid LDRD insns

> -Original Message- > From: Gcc-patches On Behalf Of Alex > Coplan > Sent: 18 May 2020 16:37 > To: Kyrylo Tkachov ; gcc-patches@gcc.gnu.org > Cc: Richard Earnshaw ; nd ; Ramana > Radhakrishnan > Subject: RE: [PATCH] [arm] Don't generate invalid LDRD insns > > > -Original

Re: [PATCH 07/13] OpenACC 2.6 deep copy: libgomp parts

Hi Julian! On 2019-12-17T22:03:47-0800, Julian Brown wrote: > This part contains the libgomp runtime support for the GOMP_MAP_ATTACH and > GOMP_MAP_DETACH mapping kinds (etc.), as introduced by the front-end > patches following in this series. > --- a/libgomp/target.c > +++ b/libgomp/target.c

Re: RFA: Remove use of register keyword in libiberty.h

Hi Guys, >> include/ChangeLog >> 2020-06-25 Nick Clifton >> >> * libiberty.h (bsearch_r): Remove use of the register keyword from >> the prototype. >> >> libiberty/ChangeLog >> 2020-06-25 Nick Clifton >> >> * bsearch.c (bsearch): Remove use of register keyword. >> *

Re: [PATCH] libgomp: added simple functions and tests for OMPD

Thanks, just nits, no need to repost, just commit it after you make those changes. On Thu, Jun 25, 2020 at 05:58:23PM -0400, y2s1982 via Gcc-patches wrote: > * Makefile.am(toolexeclib_LTLIBRARIES): Add libgompd.la. Missing space between am and (. > (libgompd_la_LDFLAGS,

[PATCH] tree-optimization/95897 - fix fold-left SLP reduction insert place

This fixes computation of the insertion place for fold-left SLP reductions where the PHIs do not have vectorized stmts. The SLP representation isn't perfect here thus the following. Bootstrap and regtest running on x86_64-unknown-linux-gnu. 2020-06-26 Richard Biener PR

Re: [PATCH] rs6000: Add support for __builtin_cpu_is ("power10")

On Thu, Jun 25, 2020 at 06:54:26PM -0500, Segher Boessenkool wrote: > Hi! > > On Thu, Jun 25, 2020 at 06:36:51PM -0500, Peter Bergner wrote: > > rs6000: Add support for __builtin_cpu_is ("power10") > > > > Add support for __builtin_cpu_is ("power10"). Also add documentation for > > the recently

Re: [PATCH] libgomp, fortran: Apply if clause to all sub-constructs in combined OpenMP constructs

On Thu, 25 Jun 2020 at 15:24, Kwok Cheung Yeung wrote: > > On 24/06/2020 6:29 pm, Tobias Burnus wrote: > > Hi Kwok, > > > > the TODO is fixed by the attached patch; I would be happy if you could > > handle > > this patch, > > e.g. together with your patch – or as follow up. > > > > (Lightly