Re: [PATCH] libstdc++: Update {x86_64, i?86, powerpc64, s390x, aarch64}-linux baselines for GCC 10.1

On 07/05/20 00:22 +0200, Eric Botcazou wrote: Oops, here are the updates from Fedora packages built during the weekend. The SPARC64/Linux bits are attached. OK for trunk and gcc-10? OK, pending RM approval for gcc-10, thanks. It looks like gcc-9 needs the same update too.

Re: [PATCH 0/4] rs6000: setbnc and friends [pu]

On 5/6/20 6:48 PM, Segher Boessenkool wrote: On Wed, May 06, 2020 at 03:41:35PM -0500, Bill Schmidt wrote: For all of these, I forgot to mention that they have been bootstrapped and tested on powerpc64le-unknown-linux-gnu with no regressions.  Are these okay for trunk, after GCC 10 is fully rele

Re: [PATCH] False-positive -Warray-bounds for char[1] on a non-zero offset in a referenced buffer

On 5/6/20 4:35 PM, Joey Liu via Gcc-patches wrote: Hi all, I think the assumption in builtin_memref::set_base_and_offset (gimple-ssa-warn-restrict.c) is not correct. It seems multiple levels of indirection may confuse gcc. When it loses the reference to the original buffer (e.g.the buffer is all

Re: [PATCH 0/4] rs6000: setbnc and friends [pu]

On Wed, May 06, 2020 at 03:41:35PM -0500, Bill Schmidt wrote: > For all of these, I forgot to mention that they have been bootstrapped > and tested on powerpc64le-unknown-linux-gnu with no regressions.  Are > these okay for trunk, after GCC 10 is fully released? These all look fine to me. But m

Re: [PATCH 1/4] rs6000: New insns setbc and setbcr

Hi! On Wed, May 06, 2020 at 03:31:08PM -0500, Bill Schmidt wrote: > (ne3): Replace :P with :GPR; use setbc for TARGET_FUTURE; > else for non-Pmode, use gen_eq and gen_xor. Before this patch, there was only ne:P, which results in the same thing (done by generic code). I should have do

[PATCH] False-positive -Warray-bounds for char[1] on a non-zero offset in a referenced buffer

Hi all, I think the assumption in builtin_memref::set_base_and_offset (gimple-ssa-warn-restrict.c) is not correct. It seems multiple levels of indirection may confuse gcc. When it loses the reference to the original buffer (e.g.the buffer is allocated by one of the parent class but it's used by a

Re: [PATCH] libstdc++: Update {x86_64, i?86, powerpc64, s390x, aarch64}-linux baselines for GCC 10.1

> Oops, here are the updates from Fedora packages built during the weekend. The SPARC64/Linux bits are attached. OK for trunk and gcc-10? 2020-05-06 Eric Botcazou * config/abi/post/sparc64-linux-gnu/baseline_symbols.txt: Update. * config/abi/post/sparc64-linux-gnu/32/baselin

[committed] d: Fix ICE in verify_gimple_stmt, at tree-cfg.c:4959

Hi, This patch removes all related helper functions around BIND_EXPR generation in the D front-end, which were the cause of an ICE. Both array concat and array new expressions wrapped any temporaries created into a BIND_EXPR. This does not work if an expression used to construct the result requ

[PATCH] c++: Missing SFINAE with lookup_fnfields [PR78446]

Here we're failing to do SFINAE in build_op_call when looking up the class's operator() via lookup_fnfields, which calls lookup_member always with complain=tf_warning_or_error. And from there we complain about an ambiguous lookup for operator(). This patch fixes this by adding a tsubst_flags_t pa

Re: [PATCH] c++: Avoid strict_aliasing_warning on dependent types or expressions [PR94951]

On 5/5/20 6:28 PM, Jakub Jelinek wrote: Hi! The following testcase gets a bogus warning during build_base_path, when cp_build_indirect_ref* calls strict_aliasing_warning with a dependent expression. IMHO calling get_alias_set etc. on dependent types feels wrong to me, we should just defer the w

Re: [PATCH] libgcc: aarch64: Get hwcap for FreeBSD

On Wed, 6 May 2020, Andreas Tobler wrote: > +#ifndef __FreeBSD__ >unsigned long hwcap = __getauxval (AT_HWCAP); > +#else > + unsigned long hwcap; Would it make sense to change the logic to #ifdef __FreeBSD__ .. #else .. #endif ? I believe that makes it easier to potentially exten

Re: [PATCH v2] c++: ICE in value_dependent_expression_p in C++98 mode [PR94938]

On 5/5/20 5:35 PM, Marek Polacek wrote: On Mon, May 04, 2020 at 09:18:57PM -0400, Jason Merrill wrote: On 5/4/20 7:32 PM, Marek Polacek wrote: Here we ICE with -std=c++98 since the newly added call to uses_template_parms (r10-6357): we hit 26530 gcc_assert (cxx_dialect >= cxx11 2653

Re: [PATCH] c++: ICE with -Wall and constexpr if [PR94937]

On 5/5/20 6:17 PM, Marek Polacek wrote: An ICE arises here because we call cp_get_callee_fndecl_nofold in a template, and we've got a CALL_EXPR whose CALL_EXPR_FN is a BASELINK. This tickles the INDIRECT_TYPE_P assert in cp_get_fndecl_from_callee. Jakub said in the PR that he'd hit a similar pro

Re: [PATCH] c++: Don't synthesize sfk_comparison method multiple times [PR94907]

On 5/5/20 3:15 AM, Jakub Jelinek wrote: Hi! On the following testcase we ICE, because synthesize_method is called twice on the same sfk_comparison method fndecl, the first time it works fine because start_preparsed_function in that case sets both current_function_decl and cfun, but second time i

[committed] i386: Use generic division to generate INEXACT exception

Introduce math_force_eval_div to use generic division to generate INEXACT as well as INVALID and DIVZERO exceptions. libgcc/ChangeLog: 2020-05-06 Uroš Bizjak * config/i386/sfp-exceptions.c (__math_force_eval): Remove. (__math_force_eval_div): New define. (__sfp_handle_exceptions):

Re: [PATCH 0/4] rs6000: setbnc and friends [pu]

For all of these, I forgot to mention that they have been bootstrapped and tested on powerpc64le-unknown-linux-gnu with no regressions.  Are these okay for trunk, after GCC 10 is fully released? Thanks, Bill On 5/6/20 3:31 PM, Bill Schmidt via Gcc-patches wrote: *** BLURB HERE *** Bill Schmi

Re: [PATCH] testsuite:analyzer: Fix header include for FreeBSD

On 06.05.20 22:27, Kamil Rytarowski wrote: On 06.05.2020 22:25, Andreas Tobler wrote: On 06.05.20 22:12, Jakub Jelinek wrote: On Wed, May 06, 2020 at 09:54:47PM +0200, Andreas Tobler wrote: --- a/gcc/testsuite/gcc.dg/analyzer/alloca-leak.c +++ b/gcc/testsuite/gcc.dg/analyzer/alloca-leak.c @@ -

[PATCH 0/4] rs6000: setbnc and friends [pu]

*** BLURB HERE *** Bill Schmidt (4): Add insns for setbc and setbcr Add tests for setbc and setbcr Add insns for setnbc and setnbcr Add tests for setnbc and setnbcr gcc/config/rs6000/rs6000.md | 100 +--- gcc/testsuite/gcc.target/powerpc/setbc.h| 27 +

Re: [committed] combine: Don't replace SET_SRC with REG_EQUAL note content if SET_SRC has side-effects [PR94873]

On Wed, May 06, 2020 at 09:36:55AM +0200, Jakub Jelinek wrote: > There were some discussions about whether REG_EQUAL notes are valid on insns > with a single > set which contains auto-inc-dec side-effects in the SET_SRC and the majority > thinks that > it should be valid. So, this patch fixes th

[PATCH 4/4] rs6000: Tests for setnbc

2020-05-06 Segher Boessenkool * gcc.target/powerpc/setnbc.h: New. * gcc.target/powerpc/setnbceq.c: New. * gcc.target/powerpc/setnbcge.c: New. * gcc.target/powerpc/setnbcgt.c: New. * gcc.target/powerpc/setnbcle.c: New. * gcc.target/powerpc/setnbclt

[PATCH] c++: ICE when shortening right shift [PR94955]

Since r10-6527 fold_for_warn calls maybe_constant_value, which means it can fold more than it previously could. In this testcase it means that cp_build_binary_op/RSHIFT_EXPR set short_shift because now we were able to fold op1 to an INTEGER_CST. But then when actually performing the shortening we

[PATCH 2/4] rs6000: Tests for setbc

2020-05-06 Segher Boessenkool * gcc.target/powerpc/setbc.h: New. * gcc.target/powerpc/setbceq.c: New. * gcc.target/powerpc/setbcge.c: New. * gcc.target/powerpc/setbcgt.c: New. * gcc.target/powerpc/setbcle.c: New. * gcc.target/powerpc/setbclt.c: Ne

[PATCH 1/4] rs6000: New insns setbc and setbcr

New instructions setbc and setbcr. setbc sets a GPR to 1 if some condition register bit is set, and 0 otherwise; setbcr does it the other way around. 2020-05-06 Segher Boessenkool * config/rs6000/rs6000.md (setbc_signed_): New define_insn. (*setbcr_signed_): Likewise.

[PATCH 3/4] rs6000: New insns setnbc and setnbcr

setnbc[r] is like setbc[r], but it writes -1 instead of 1 to the GPR. 2020-05-06 Segher Boessenkool * config/rs6000/rs6000.md (*setnbc_signed_): New define_insn. (*setnbcr_signed_): New define_insn. (*neg_eq_): Avoid for TARGET_FUTURE; add missing && 1.

Re: [PATCH] testsuite:analyzer: Fix header include for FreeBSD

On 06.05.2020 22:25, Andreas Tobler wrote: > On 06.05.20 22:12, Jakub Jelinek wrote: >> On Wed, May 06, 2020 at 09:54:47PM +0200, Andreas Tobler wrote: >>> --- a/gcc/testsuite/gcc.dg/analyzer/alloca-leak.c >>> +++ b/gcc/testsuite/gcc.dg/analyzer/alloca-leak.c >>> @@ -1,5 +1,6 @@ >>> -#include >>>

Re: [PATCH] testsuite:analyzer: Fix header include for FreeBSD

On 06.05.20 22:12, Jakub Jelinek wrote: On Wed, May 06, 2020 at 09:54:47PM +0200, Andreas Tobler wrote: --- a/gcc/testsuite/gcc.dg/analyzer/alloca-leak.c +++ b/gcc/testsuite/gcc.dg/analyzer/alloca-leak.c @@ -1,5 +1,6 @@ -#include - +#include It needs to be: +#if __has_include() +#include

[PATCH] libgcc: aarch64: Get hwcap for FreeBSD

Hi all, Since FreeBSD 12, FreeBSD has a sys/auxv.h header too but it doesn't provide the getauxval function. Instead it offers the elf_aux_info function which provides a similar functionality. This patch gets the hwcap for FreeBSD. Is this ok for trunk? TIA, Andreas +2020-05-05 Andreas Tobler

Re: [PATCH] testsuite:analyzer: Fix header include for FreeBSD

On Wed, May 06, 2020 at 09:54:47PM +0200, Andreas Tobler wrote: > --- a/gcc/testsuite/gcc.dg/analyzer/alloca-leak.c > +++ b/gcc/testsuite/gcc.dg/analyzer/alloca-leak.c > @@ -1,5 +1,6 @@ > -#include > - > +#include It needs to be: > +#if __has_include() +#include > +#endif __has_include doesn

Re: [PATCH] testsuite:analyzer: Fix header include for FreeBSD

On 04.05.20 12:10, Kamil Rytarowski wrote: On 04.05.2020 12:04, Jakub Jelinek wrote: On Mon, May 04, 2020 at 11:49:27AM +0200, Kamil Rytarowski wrote: Please include in your patch "|| defined(__NetBSD__)". is this ok for you? This is one reason why I'd prefer #define alloca __builtin_alloca

Re: [PATCH] coroutines: Improve error recovery [PR94817, PR94829].

On 4/29/20 6:50 AM, Iain Sandoe wrote: Hi, When we have completely missing key information (e.g. the coroutine_traits) or a partially transformed function body, we need to try and balance returning useful information about failures with the possibility that some part of the diagnostics machinery

Re: [PATCH] Improve std::fill for vector

On 06/05/20 20:46 +0200, François Dumont via Libstdc++ wrote: Hi I am not clear about current stage so I am proposing this trivial patch to find out if we are back in stage 1. The current status is always shown on the front page of gcc.gnu.org (although currently the link to the GCC 11 status

Re: [PATCH] libstdc++: Update Solaris baselines for GCC 9.4

On 06/05/20 20:33 +0200, Rainer Orth wrote: Hi Jonathan, On 06/05/20 14:12 +0200, Rainer Orth wrote: Hi Jonathan, On 06/05/20 10:49 +0200, Rainer Orth wrote: I just remembered that the libstdc++ ABI baselines haven't been updated for the GCC 10 release yet. This patch corrects this for Sol

[pushed] coroutines: Remove references to n4849 (NFC).

Hi, Another minor cleanup. This just strips out references to the draft standard numbered n4849. The implementation is now intended to be applicable to the expected final version. tested on x86_64-darwin16, applied to master as obvious thanks Iain gcc/cp/ChangeLog: 2020-05-05 Iain Sandoe

Re: PING[STAGE 1][PATCH][x86][1/3]: Add -mzero-caller-saved-regs=[skip|used-gpr|all-gpr|used|all]

Hi, Kees, > On May 4, 2020, at 1:21 PM, Kees Cook wrote: > > On Mon, May 04, 2020 at 11:51:49AM -0500, Qing Zhao wrote: >> Hi, >> >> This is a PING for this patch for gcc11 stage 1. >> >> https://gcc.gnu.org/pipermail/gcc-patches/2020-April/544058.html >>

[PATCH] Improve std::fill for vector

Hi I am not clear about current stage so I am proposing this trivial patch to find out if we are back in stage 1. This patch extend the overload so that it is used even when _GLIBCXX_DEBUG mode is activated.     * include/bits/stl_algobase.h (struct _Bit_iterator): New declaration.

Committed [Version 3][PATCH][gcc][PR94230]provide an option to change the size limitation for -Wmisleading-indent

FYI. > On Apr 23, 2020, at 5:13 PM, David Malcolm wrote: > > May need updating to match the hint message. > > [...] > > This is OK for stage 1 with those nits fixed. I committed the updated patch with all the suggestions today to gcc11 stage1 as: https://gcc.gnu.org/g:530b44094354758d0dea537

[PATCH] libstdc++: Update Solaris baselines for GCC 9.4

Hi Jonathan, > On 06/05/20 14:12 +0200, Rainer Orth wrote: >>Hi Jonathan, >> >>> On 06/05/20 10:49 +0200, Rainer Orth wrote: I just remembered that the libstdc++ ABI baselines haven't been updated for the GCC 10 release yet. This patch corrects this for Solaris/SPARC and x86. >>> >>>

Re: [PR C++/94946] ambiguous overload regression

On Wed, May 06, 2020 at 02:14:03PM -0400, Nathan Sidwell wrote: > @@ -0,0 +1,7 @@ > +// { dg-do compile { target { i?86-*-* x86_64-*-* } } } > +// { dg-options -m32 } > +// PR 94946 > +class a { > + template a(b (*)()); > + template a(b(__attribute__((fastcall)) *c)()); > +}; I think instead o

[PR C++/94946] ambiguous overload regression

This appears to fix 94946 -- regular testsuite passes, bootstrapping now. I'll commit if the bootstrap is good. Jakub, ok for gcc-10 too? nathan -- Nathan Sidwell 2020-05-06 Nathan Sidwell PR c++/94946 * decl.c (grokdeclarator): Don't splice template attributes in parm context -- they

Re: [PATCH] PowerPC -mcpu=future Patch 3 of 7, Add test for generating prefixed load/store

Hi! On Tue, May 05, 2020 at 08:41:35PM -0400, Hans-Peter Nilsson wrote: > On Fri, 1 May 2020, Segher Boessenkool wrote: > > On Mon, Apr 27, 2020 at 05:01:34PM -0500, will schmidt wrote: > > > On Mon, 2020-04-27 at 15:53 -0400, Michael Meissner via Gcc-patches wrote: > > > > +unsigned long > > > >

Re: [PATCH] lto-wrapper: split arguments of getenv ("MAKE").

On Wed, May 6, 2020 at 3:09 PM Martin Liška wrote: > > On 5/6/20 2:52 PM, Richard Biener wrote: > > On Wed, May 6, 2020 at 2:08 PM Martin Liška wrote: > >> > >> Hi. > >> > >> The patch splits arguments in getenv ("MAKE") so that one can use: > >> > >> $ MAKE="make -s" gcc main.o -flto=16 > >> > >

Re: PING[STAGE 1][PATCH][x86][1/3]: Add -mzero-caller-saved-regs=[skip|used-gpr|all-gpr|used|all]

On 5/4/20 10:51 AM, Qing Zhao via Gcc-patches wrote: Hi, This is a PING for this patch for gcc11 stage 1. https://gcc.gnu.org/pipermail/gcc-patches/2020-April/544058.html Please take a look on it. Just a couple of minor poin

[committed] i386: Use ADD to implement compares with negated operand [PR94913]

Use carry flag from addition to implement GEU/LTU compares with negated operand, so e.g. ~x < y compiles to: addq%rsi, %rdi setc%al instead of: notq%rdi cmpq%rsi, %rdi setb%al 2020-05-06 Uroš Bizjak PR target/94913 *

RE: [PATCH][AArch64] Use __getauxval instead of getauxval in LSE detection code in libgcc

> -Original Message- > From: Joseph Myers > Sent: 06 May 2020 15:46 > To: Richard Biener > Cc: Kyrylo Tkachov ; Florian Weimer > ; Szabolcs Nagy ; gcc- > patc...@gcc.gnu.org; Jakub Jelinek > Subject: Re: [PATCH][AArch64] Use __getauxval instead of getauxval in LSE > detection code in

Re: [PATCH][AArch64] Use __getauxval instead of getauxval in LSE detection code in libgcc

On Wed, 6 May 2020, Richard Biener wrote: > > Here is the updated patch for the record. > > Jakub, richi, is this ok for the GCC 10 branch? > > I'll defer to Joseph who is release manager as well. This version is OK with me. -- Joseph S. Myers jos...@codesourcery.com

Re: [PATCH] tsan: Add optional support for distinguishing volatiles

Hello, Jakub, On Tue, 28 Apr 2020 at 16:58, Dmitry Vyukov wrote: > > On Tue, Apr 28, 2020 at 4:55 PM Jakub Jelinek wrote: > > > > On Tue, Apr 28, 2020 at 04:48:31PM +0200, Dmitry Vyukov wrote: > > > FWIW this is: > > > > > > Acked-by: Dmitry Vyukov > > > > > > We just landed a similar change to

Re: [PATCH] c++: template instantiation during fold_for_warn [PR94038]

On Wed, 6 May 2020, Patrick Palka wrote: > On Tue, 5 May 2020, Patrick Palka wrote: > > > On Tue, 5 May 2020, Patrick Palka wrote: > > > > > Unfortunately, the previous fix to PR94038 is fragile. When the > > > argument to fold_for_warn is a bare CALL_EXPR, then all is well: the > > > result of

[PATCH] Prepare removal of SLP_INSTANCE_GROUP_SIZE

This removes trivial instances of SLP_INSTANCE_GROUP_SIZE and refrains from using a "SLP instance" which nowadays is just one of the possibly many entries into the SLP graph. Bootstrapped / tested on x86_64-unknown-linux-gnu, pushed. Richard. 2020-05-06 Richard Biener * tree-vectoriz

Re: [PATCH][AArch64] Use __getauxval instead of getauxval in LSE detection code in libgcc

The 05/06/2020 11:39, Kyrylo Tkachov wrote: > > -Original Message- > > From: Florian Weimer > > Sent: 06 May 2020 11:28 > > > > Is __gnu_linux__ defined with musl? It's not really GNU/Linux, after > > all. musl doesn't have __getauxval IIRC, so it would break the build > > there. > > I

Re: [PATCH] x32: Update baseline_symbols.txt

On 06/05/20 05:37 -0700, H.J. Lu via Libstdc++ wrote: On Wed, May 6, 2020 at 2:39 AM Jonathan Wakely via Gcc-patches wrote: On 06/05/20 10:15 +0100, Jonathan Wakely wrote: >On 06/05/20 11:09 +0200, Jakub Jelinek via Libstdc++ wrote: >>On Wed, May 06, 2020 at 10:49:13AM +0200, Rainer Orth wrote

Re: [PATCH] tree-optimization/57359 - rewrite SM code

On Tue, 5 May 2020, Richard Biener wrote: > > This rewrites store-motion to process candidates where we can > ensure order preserving separately and with no need to disambiguate > against all stores. Those candidates we cannot handle this way > are validated to be independent on all stores (w/o

Re: [PATCH] x32: Update baseline_symbols.txt

On Wed, May 06, 2020 at 02:14:58PM +0100, Jonathan Wakely wrote: > > Here is the x32 part. OK for master and GCC 10 branch? > > OK, subject to RM approval for gcc-10 branch. Ok for 10.1. Jakub

Re: [PATCH] libstdc++: Update Solaris baselines for GCC 10.1

On 06/05/20 14:12 +0200, Rainer Orth wrote: Hi Jonathan, On 06/05/20 10:49 +0200, Rainer Orth wrote: I just remembered that the libstdc++ ABI baselines haven't been updated for the GCC 10 release yet. This patch corrects this for Solaris/SPARC and x86. I think we also need to update them on

Re: [PATCH] lto-wrapper: split arguments of getenv ("MAKE").

On 5/6/20 2:52 PM, Richard Biener wrote: On Wed, May 6, 2020 at 2:08 PM Martin Liška wrote: Hi. The patch splits arguments in getenv ("MAKE") so that one can use: $ MAKE="make -s" gcc main.o -flto=16 Right now it fails due to: [pid 6004] execve("/home/marxin/Programming/script-misc/make -s

Re: [PATCH] lto-wrapper: split arguments of getenv ("MAKE").

On Wed, May 6, 2020 at 2:08 PM Martin Liška wrote: > > Hi. > > The patch splits arguments in getenv ("MAKE") so that one can use: > > $ MAKE="make -s" gcc main.o -flto=16 > > Right now it fails due to: > [pid 6004] execve("/home/marxin/Programming/script-misc/make -s", ["make > -s", "-f", "/tmp/

[PATCH] x32: Update baseline_symbols.txt

On Wed, May 6, 2020 at 2:39 AM Jonathan Wakely via Gcc-patches wrote: > > On 06/05/20 10:15 +0100, Jonathan Wakely wrote: > >On 06/05/20 11:09 +0200, Jakub Jelinek via Libstdc++ wrote: > >>On Wed, May 06, 2020 at 10:49:13AM +0200, Rainer Orth wrote: > >>>I just remembered that the libstdc++ ABI ba

Re: [PATCH] libstdc++: Update Solaris baselines for GCC 10.1

Hi Jonathan, > On 06/05/20 10:49 +0200, Rainer Orth wrote: >>I just remembered that the libstdc++ ABI baselines haven't been updated >>for the GCC 10 release yet. This patch corrects this for Solaris/SPARC >>and x86. > > I think we also need to update them on the gcc-9 branch, all the > 3.4.28 sy

Re: [PATCH v2] Add handling of MULT_EXPR/PLUS_EXPR for wrapping overflow in affine combination(PR83403)

On Thu, 30 Apr 2020, luoxhu wrote: > Update the patch with overflow check. Bootstrap and regression tested PASS > on Power8-LE. > > > Use determine_value_range to get value range info for fold convert expressions > with internal operation PLUS_EXPR/MINUS_EXPR/MULT_EXPR when not overflow on > w

[PATCH] lto-wrapper: split arguments of getenv ("MAKE").

Hi. The patch splits arguments in getenv ("MAKE") so that one can use: $ MAKE="make -s" gcc main.o -flto=16 Right now it fails due to: [pid 6004] execve("/home/marxin/Programming/script-misc/make -s", ["make -s", "-f", "/tmp/ccNpRBlZ.mk", "-j16", "all"], 0x4b69b0 /* 82 vars */) = -1 ENOENT (N

Re: [patch] Fix a few DWARF bugs with -fgnat-encodings=minimal

On Tue, May 5, 2020 at 10:43 AM Eric Botcazou wrote: > > Hi, > > The -fgnat-encodings=minimal switch tells the compiler to generate motly pure > DWARF for the GNAT compiler and it contains some bugs related to discriminated > record types with variant part. > > Tested on x86-64/Linux, OK for the m

Re: [PATCH] c-attribs.c: Fix use of uninitialized variable nunits

On Tue, May 5, 2020 at 5:47 PM Stefan Schulze Frielinghaus wrote: > > On Tue, May 05, 2020 at 02:58:51PM +0200, Richard Biener wrote: > > On Mon, May 4, 2020 at 4:07 PM Stefan Schulze Frielinghaus > > wrote: > > > > > > On Mon, May 04, 2020 at 03:19:02PM +0200, Richard Biener wrote: > > > > On Mo

Re: [PATCH][AArch64] Use __getauxval instead of getauxval in LSE detection code in libgcc

On Wed, May 6, 2020 at 1:56 PM Kyrylo Tkachov wrote: > > > > > -Original Message- > > From: Kyrylo Tkachov > > Sent: 06 May 2020 11:40 > > To: Florian Weimer > > Cc: Joseph Myers ; Szabolcs Nagy > > ; gcc-patches@gcc.gnu.org; Jakub Jelinek > > ; Richard Biener > > Subject: RE: [PATCH][AA

Re: [PATCH] tree-ssa-structalias.c: Fix comments

On Wed, May 6, 2020 at 1:07 PM Erick Ochoa wrote: > > This patch fixes some quotations inside comments. The change in syntax > highlighting was bothering me. I also found a typo. OK. > ChangeLog: > > 2020-05-06 Erick Ochoa > > * gcc/tree-ssa-struct-alias.c: Fix comments > > > diff --gi

RE: [PATCH][AArch64] Use __getauxval instead of getauxval in LSE detection code in libgcc

> -Original Message- > From: Kyrylo Tkachov > Sent: 06 May 2020 11:40 > To: Florian Weimer > Cc: Joseph Myers ; Szabolcs Nagy > ; gcc-patches@gcc.gnu.org; Jakub Jelinek > ; Richard Biener > Subject: RE: [PATCH][AArch64] Use __getauxval instead of getauxval in LSE > detection code in lib

Re: [PATCH] aarch64: fix conflicting declarations

Andreas Schwab writes: > aarch64_get_extension_string_for_isa_flags is declared in > "aarch64-protos.h", use that instead of re-declaring it improperly. > > * config/aarch64/driver-aarch64.c: Include "aarch64-protos.h". > (aarch64_get_extension_string_for_isa_flags): Don't declare. OK

[PATCH] tree-ssa-structalias.c: Fix comments

This patch fixes some quotations inside comments. The change in syntax highlighting was bothering me. I also found a typo. ChangeLog: 2020-05-06 Erick Ochoa * gcc/tree-ssa-struct-alias.c: Fix comments diff --git a/gcc/tree-ssa-structalias.c b/gcc/tree-ssa-structalias.c index 416a26

RE: [PATCH][AArch64] Use __getauxval instead of getauxval in LSE detection code in libgcc

> -Original Message- > From: Florian Weimer > Sent: 06 May 2020 11:28 > To: Kyrylo Tkachov > Cc: Joseph Myers ; Szabolcs Nagy > ; gcc-patches@gcc.gnu.org; Jakub Jelinek > ; Richard Biener > Subject: Re: [PATCH][AArch64] Use __getauxval instead of getauxval in LSE > detection code in l

[PATCH] middle-end/94964 - avoid EH loop entry with CP_SIMPLE_PREHEADERS

Loop optimizers expect to be able to insert on the preheader edge w/o splitting it thus avoid ending up with a preheader that enters the loop via an EH edge (or an abnormal edge). Bootstrapped on x86_64-unknown-linux-gnu, testing in progress. Richard. 2020-05-06 Richard Biener PR m

[PATCH] tree-optimization/94963 - avoid bogus uninit warning with store-motion

Eliding the load for store-motion causes an uninitialized variable flowing into the loop, conditionally initialized and used. The uninit warning cannot relate the flag used to guard the initialization and use with the actual initialization so the following robustifies the previous approach of ma

Re: [PATCH] aarch64: prefer using csinv, csneg in zero extend contexts

Alex Coplan writes: >> -Original Message- >> From: Richard Sandiford >> Sent: 30 April 2020 15:13 >> To: Alex Coplan >> Cc: gcc-patches@gcc.gnu.org; Richard Earnshaw ; >> Marcus Shawcroft ; Kyrylo Tkachov >> ; nd >> Subject: Re: [PATCH] aarch64: prefer using csinv, csneg in zero extend

Re: [PATCH][AArch64] Use __getauxval instead of getauxval in LSE detection code in libgcc

* Kyrylo Tkachov: > diff --git a/libgcc/config/aarch64/lse-init.c > b/libgcc/config/aarch64/lse-init.c > index 74acef25cce..57236610468 100644 > --- a/libgcc/config/aarch64/lse-init.c > +++ b/libgcc/config/aarch64/lse-init.c > @@ -29,19 +29,20 @@ see the files COPYING3 and COPYING.RUNTIME respect

Re: [PATCH 4/4] Use const for template argument.

On 04/02/20 14:55 +0100, Martin Liska wrote: diff --git a/libstdc++-v3/include/parallel/multiway_merge.h b/libstdc++-v3/include/parallel/multiway_merge.h index 983c7b2bd9a..97a9ce0feb7 100644 --- a/libstdc++-v3/include/parallel/multiway_merge.h +++ b/libstdc++-v3/include/parallel/multiway_merge.

RE: [PATCH][AArch64] Use __getauxval instead of getauxval in LSE detection code in libgcc

> -Original Message- > From: Joseph Myers > Sent: 05 May 2020 22:30 > To: Kyrylo Tkachov > Cc: Szabolcs Nagy ; Florian Weimer > ; gcc-patches@gcc.gnu.org > Subject: RE: [PATCH][AArch64] Use __getauxval instead of getauxval in LSE > detection code in libgcc > > On Tue, 5 May 2020, Kyryl

Re: [PATCH] Add enqcmd, avx512bf16, avx512vp2intersect to funcspec-56.inc

On Wed, May 6, 2020 at 11:08 AM Hongtao Liu wrote: > > Hi: > Test is ok for funcspec-5.c, funcspec-6.c. > > gcc/testuite/ChangeLog > * gcc.target/i386/funcspec-56.inc: Add enqcmd, avx512bf16, > avx512vp2intersect. OK. Thanks, Uros. > gcc/testsuite/gcc.target/i386/funcspec-56.

Re: [PATCH] libstdc++: Update {x86_64,i?86,powerpc64,s390x,aarch64}-linux baselines for GCC 10.1

On 06/05/20 10:15 +0100, Jonathan Wakely wrote: On 06/05/20 11:09 +0200, Jakub Jelinek via Libstdc++ wrote: On Wed, May 06, 2020 at 10:49:13AM +0200, Rainer Orth wrote: I just remembered that the libstdc++ ABI baselines haven't been updated for the GCC 10 release yet. This patch corrects this

Re: [PATCH] libstdc++: Update {x86_64,i?86,powerpc64,s390x,aarch64}-linux baselines for GCC 10.1

On 06/05/20 11:09 +0200, Jakub Jelinek via Libstdc++ wrote: On Wed, May 06, 2020 at 10:49:13AM +0200, Rainer Orth wrote: I just remembered that the libstdc++ ABI baselines haven't been updated for the GCC 10 release yet. This patch corrects this for Solaris/SPARC and x86. Oops, here are the u

[PATCH] libstdc++: Update {x86_64,i?86,powerpc64,s390x,aarch64}-linux baselines for GCC 10.1

On Wed, May 06, 2020 at 10:49:13AM +0200, Rainer Orth wrote: > I just remembered that the libstdc++ ABI baselines haven't been updated > for the GCC 10 release yet. This patch corrects this for Solaris/SPARC > and x86. Oops, here are the updates from Fedora packages built during the weekend. Ok f

[PATCH] Add enqcmd,avx512bf16,avx512vp2intersect to funcspec-56.inc

Hi: Test is ok for funcspec-5.c, funcspec-6.c. gcc/testuite/ChangeLog * gcc.target/i386/funcspec-56.inc: Add enqcmd, avx512bf16, avx512vp2intersect. gcc/testsuite/gcc.target/i386/funcspec-56.inc | 6 ++ 1 file changed, 6 insertions(+) diff --git a/gcc/testsuite/gcc.target/

Re: [PATCH] match.pd: Optimize ~(~X +- Y) into (X -+ Y) [PR94921]

On Wed, 6 May 2020, Jakub Jelinek wrote: > Hi! > > According to my verification proglet, this transformation for signed types > with undefined overflow doesn't introduce nor remove any UB cases, so should > be valid even for signed integral types. > Not using a for because of the :c on plus which

Re: [PATCH] libstdc++: Update Solaris baselines for GCC 10.1

On Wed, May 06, 2020 at 10:00:02AM +0100, Jonathan Wakely wrote: > On 06/05/20 10:49 +0200, Rainer Orth wrote: > > I just remembered that the libstdc++ ABI baselines haven't been updated > > for the GCC 10 release yet. This patch corrects this for Solaris/SPARC > > and x86. > > I think we also ne

Re: [PATCH] libstdc++: Update Solaris baselines for GCC 10.1

On 06/05/20 10:49 +0200, Rainer Orth wrote: I just remembered that the libstdc++ ABI baselines haven't been updated for the GCC 10 release yet. This patch corrects this for Solaris/SPARC and x86. I think we also need to update them on the gcc-9 branch, all the 3.4.28 symbols are on that branch

RE: [PATCH PR94026] combine missed opportunity to simplify comparisons with zero

Hi, > -Original Message- > From: Segher Boessenkool [mailto:seg...@kernel.crashing.org] > Sent: Tuesday, March 24, 2020 10:58 PM > To: Yangfei (Felix) > Cc: gcc-patches@gcc.gnu.org; Zhanghaijian (A) > Subject: Re: [PATCH PR94026] combine missed opportunity to simplify > comparisons with

[PATCH] libstdc++: Update Solaris baselines for GCC 10.1

I just remembered that the libstdc++ ABI baselines haven't been updated for the GCC 10 release yet. This patch corrects this for Solaris/SPARC and x86. Created on master with make new-abi-baseline on i386-pc-solaris2.11 and sparc-sun-solaris2.11. Compared so far with last Friday's gcc-10 branch

[PATCH] tree-optimization/94965 - fix typo in vec_info * passing

Should have passed vinfo, not loop_vinfo. Pushed as obvious. 2020-05-06 Richard Biener PR tree-optimization/94965 * tree-vect-stmts.c (vectorizable_load): Fix typo. --- gcc/tree-vect-stmts.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/gcc/tree-vect-s

Re: [committed] riscv: Fix up riscv_atomic_assign_expand_fenv [PR94950]

Thank you! On Wed, May 6, 2020 at 12:43 AM Jakub Jelinek wrote: > > Hi! > > Similarly to the fixes on many other targets, riscv needs to use TARGET_EXPR > to avoid having the create_tmp_var_raw temporaries without proper DECL_CONTEXT > and not mentioned in local decls. > > Committed as obvious t

[PATCH] match.pd: Optimize ~(~X +- Y) into (X -+ Y) [PR94921]

Hi! According to my verification proglet, this transformation for signed types with undefined overflow doesn't introduce nor remove any UB cases, so should be valid even for signed integral types. Not using a for because of the :c on plus which can't be there on minus. Bootstrapped/regtested on x

[committed] riscv: Fix up riscv_atomic_assign_expand_fenv [PR94950]

Hi! Similarly to the fixes on many other targets, riscv needs to use TARGET_EXPR to avoid having the create_tmp_var_raw temporaries without proper DECL_CONTEXT and not mentioned in local decls. Committed as obvious to trunk. 2020-05-06 Jakub Jelinek PR target/94950 * config/r

[committed] combine: Don't replace SET_SRC with REG_EQUAL note content if SET_SRC has side-effects [PR94873]

Hi! There were some discussions about whether REG_EQUAL notes are valid on insns with a single set which contains auto-inc-dec side-effects in the SET_SRC and the majority thinks that it should be valid. So, this patch fixes the combiner to punt in that case, because otherwise the auto-inc-dec