Re: [PATCH] Change march=alderlake ISA list and add m_ALDERLAKE to m_CORE_AVX2

On Mon, Apr 12, 2021 at 3:20 PM Uros Bizjak via Gcc-patches wrote: > > On Mon, Apr 12, 2021 at 5:13 AM Cui, Lili wrote: > > > > Hi Uros, > > > > This patch is about to change Alder Lake ISA list to GCC add m_ALDERLAKE to > > m_CORE_AVX2. > > Alder Lake Intel Hybrid Technology is based on Tremont

Re: [PATCH] testsuite/arm: Fix scan-assembler-times in pr96770.c with movt/movw

ping? On Tue, 6 Apr 2021 at 14:02, Christophe Lyon wrote: > > ping? > > On Mon, 29 Mar 2021 at 11:01, Christophe Lyon > wrote: > > > > The previous change to this testcase missed the fact that the data may > > be accessed via an anchor, depending on the optimization level, > > leading to false f

[ping] Fix thinko in libcpp preparation patch for modules

This is a regression in Ada on the mainline, 10 and 9 branches: https://gcc.gnu.org/pipermail/gcc-patches/2021-March/567143.html Thanks in advance. -- Eric Botcazou

Re: [GCC 12] [PATCH] Add inline_ignore_target function attribute

On Sat, Apr 10, 2021 at 5:11 PM H.J. Lu via Gcc-patches wrote: > > Add inline_ignore_target function attribute to inform the compiler that > target specific option mismatch on functions with the always_inline > attribute may be ignored. On x86 targets, this attribute can be used on > integer func

[PATCH wwwdoc] Mention Rocketlake [GCC11]

Updated wwwdocs for Rocketlake [GCC11], thanks. [PATCH] Mention Rocketlake --- htdocs/gcc-11/changes.html | 4 1 file changed, 4 insertions(+) diff --git a/htdocs/gcc-11/changes.html b/htdocs/gcc-11/changes.html index a7fa4e1b..38725abc 100644 --- a/htdocs/gcc-11/changes.html +++ b/htdocs

Re: [GCC 9] [PR47785] COLLECT_AS_OPTIONS

On Fri, Apr 9, 2021 at 10:16 PM H.J. Lu wrote: > > On Sun, Feb 23, 2020 at 10:32 PM Prathamesh Kulkarni > wrote: > > > > On Wed, 19 Feb 2020 at 19:54, Richard Biener > > wrote: > > > > > > > > > > > OK with those changes. Did you try if the diagnostics are visible > > > > > (when you add -Wl,-

[committed] libstdc++: Fix test that fails in C++20 mode [PR 99995]

The 17_intro/headers/c++1998/49745.cc test fails for C++20 mode with PCH enabled, because PCH makes it include , which includes , and that includes in C++20 mode. The dependency should go away when C++20 atomic waiting is stable, but will probably remain while the feature is experimental. Change

[PATCH][pushed] mklog: support long filenames

Pushed to master. Martin contrib/ChangeLog: * gcc-changelog/git_commit.py: Support long filenames in entries. * gcc-changelog/test_email.py: Test it. * gcc-changelog/test_patches.txt: Likewise. --- contrib/gcc-changelog/git_commit.py| 6 +++-- contrib/gcc-ch

[PATCH] ASAN: do not unpoison in OpenMP context

Hello. Right now, we do not allow ASAN poisoning/unpoisoning for auto variables if we are in a gimplify_omp_ctxp context. That's fine, but also need to omit emission of unpoison calls when there's a goto jump pointing from OMP context. Patch can bootstrap on x86_64-linux-gnu and survives regressi

Re: [committed] libstdc++: Fix test that fails in C++20 mode [PR 99995]

And this one fixes a couple more tests that fail with C++20. Tested x86_64-linux. Committed to trunk. commit 91dd7954c44b201aa3b16e66b18d7a5df00fded9 Author: Jonathan Wakely Date: Mon Apr 12 11:45:21 2021 libstdc++: Fix some tests that fail in C++20 mode The linear_congruential

Re: [PATCH] ASAN: do not unpoison in OpenMP context

On Mon, Apr 12, 2021 at 01:21:29PM +0200, Martin Liška wrote: > gcc/ChangeLog: > > PR sanitizer/99877 > * gimplify.c (gimplify_expr): Right now, we unpoison all > variables before a goto . We should not do it if we are > in a omp context. > > gcc/testsuite/ChangeLog: > >

[PATCH] docs: update symver attribute description

This improves documentation as noticed by Jakub. Ready for master? Thanks, Martin gcc/ChangeLog: * doc/extend.texi: Be more precise in documentation of symver attribute. --- gcc/doc/extend.texi | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/gcc/doc/ex

Re: [PATCH] docs: update symver attribute description

On Mon, Apr 12, 2021 at 01:44:54PM +0200, Martin Liška wrote: > This improves documentation as noticed by Jakub. > > Ready for master? > Thanks, > Martin > > gcc/ChangeLog: > > * doc/extend.texi: Be more precise in documentation > of symver attribute. Ok, but I'd prefer to see the o

Re: [ping] Fix thinko in libcpp preparation patch for modules

On 4/12/2021 2:45 AM, Eric Botcazou wrote: This is a regression in Ada on the mainline, 10 and 9 branches: https://gcc.gnu.org/pipermail/gcc-patches/2021-March/567143.html Thanks in advance. OK jeff

Re: [PATCH] docs: update symver attribute description

On 4/12/21 1:50 PM, Jakub Jelinek wrote: > On Mon, Apr 12, 2021 at 01:44:54PM +0200, Martin Liška wrote: >> This improves documentation as noticed by Jakub. >> >> Ready for master? >> Thanks, >> Martin >> >> gcc/ChangeLog: >> >> * doc/extend.texi: Be more precise in documentation >> of sy

Re: [PATCH] docs: update symver attribute description

On Mon, Apr 12, 2021 at 02:15:16PM +0200, Martin Liška wrote: > The old syntax with the alias is quite ugly.. Not that much. And users have no other option (besides inline asm but that doesn't work with LTO well). > > so that people who don't have gcc configured against > > binutils 2.35 or newe

[committed] libstdc++: Implement LWG 3404 for C++20 subrange [PR 100044]

These deduction guides became useless with LWG 3282 (implemented in commit r10-6741) and so were removed by LWG 3404. libstdc++-v3/ChangeLog: PR libstdc++/100044 * include/bits/ranges_util.h (__detail::__iterator_sentinel_pair): Remove helper concept. (subrange(_Pr

Re: [PATCH] docs: update symver attribute description

On 4/12/21 2:18 PM, Jakub Jelinek wrote: > On Mon, Apr 12, 2021 at 02:15:16PM +0200, Martin Liška wrote: >> The old syntax with the alias is quite ugly.. > > Not that much. And users have no other option (besides inline asm > but that doesn't work with LTO well). > >>> so that people who don't h

Re: [PATCH] docs: update symver attribute description

On Mon, Apr 12, 2021 at 02:32:35PM +0200, Martin Liška wrote: > +If you have an older release of binutils release, then symbol alias needs to s/binutils release/binutils/ > +be used: > + > +@smallexample > +__attribute__ ((__symver__ ("foo@@VERS_2"))) > +__attribute__ ((alias ("foo_v1"))) > +int

[GCC][Patch] arm: Fix the mve multilib for the broken cmse support (pr99939).

Hi, The current CMSE support in the multilib build for "-march=armv8.1-m.main+mve -mfloat-abi=hard -mfpu=auto" is broken as specified in PR99939 and this patch fixes the issue. Regression tested on arm-none-eabi and found no regressions. Ok for master? and Ok for GCC-10 branch? Regards, Srinat

Re: [Patch, fortran] PR fortran/100018 - ICE on missing polymorphic argument

On 10.04.21 22:45, José Rui Faustino de Sousa via Fortran wrote: On 10/04/21 17:37, Tobias Burnus wrote: And you need an additional single-line summary for git – which should be part of the patch submission. In case you are waiting for me, I did write: 'LGTM – Thanks for the patch. Two nits:'

Re: [PATCH] docs: update symver attribute description

On 4/12/21 2:45 PM, Jakub Jelinek wrote: > On Mon, Apr 12, 2021 at 02:32:35PM +0200, Martin Liška wrote: >> +If you have an older release of binutils release, then symbol alias needs to > > s/binutils release/binutils/ Fixed. > >> +be used: >> + >> +@smallexample >> +__attribute__ ((__symver__

Re: [PATCH] docs: update symver attribute description

On Mon, Apr 12, 2021 at 03:24:00PM +0200, Martin Liška wrote: > On 4/12/21 2:45 PM, Jakub Jelinek wrote: > > On Mon, Apr 12, 2021 at 02:32:35PM +0200, Martin Liška wrote: > >> +If you have an older release of binutils release, then symbol alias needs > >> to > > > > s/binutils release/binutils/ >

[PATCH] IBM Z: Add alternative to *movdi_{31, 64} in order to load a DFP zero

Bootstraped and regtested on IBM Z. Ok for mainline? gcc/ChangeLog: * config/s390/s390.md ("*movdi_31", "*movdi_64"): Add alternative in order to load a DFP zero. --- gcc/config/s390/s390.md | 25 ++--- 1 file changed, 14 insertions(+), 11 deletions(-) dif

Re: [PATCH] docs: update symver attribute description

On Mon, Apr 12, 2021 at 5:54 AM Martin Liška wrote: > > On 4/12/21 2:18 PM, Jakub Jelinek wrote: > > On Mon, Apr 12, 2021 at 02:15:16PM +0200, Martin Liška wrote: > >> The old syntax with the alias is quite ugly.. > > > > Not that much. And users have no other option (besides inline asm > > but t

Re: [PATCH] IBM Z: Add alternative to *movdi_{31, 64} in order to load a DFP zero

On 4/12/21 3:40 PM, Stefan Schulze Frielinghaus wrote: > Bootstraped and regtested on IBM Z. Ok for mainline? > > gcc/ChangeLog: > > * config/s390/s390.md ("*movdi_31", "*movdi_64"): Add > alternative in order to load a DFP zero. Ok. Thanks! Andreas > --- > gcc/config/s390/s390

Re: [PATCH] docs: update symver attribute description

On 4/12/21 3:50 PM, H.J. Lu wrote: > GCC failed to bootstrap: > > /export/gnu/import/git/sources/gcc/gcc/doc/extend.texi:3870: misplaced { > /export/gnu/import/git/sources/gcc/gcc/doc/extend.texi:3872: misplaced } > /export/gnu/import/git/sources/gcc/gcc/doc/extend.texi:3874: unknown > command `VE

[PATCH] docs: Remove empty table column.

A column with empty values seems suspicious. Ready to be installed? Thanks, Martin gcc/fortran/ChangeLog: * intrinsic.texi: The table has first column empty and it makes trouble when processing makeinfo --xml output. --- gcc/fortran/intrinsic.texi | 18 +- 1 file

[r11-8133 Regression] Failed to bootstrap on Linux/x86_64

On Linux/x86_64, bb8f2c32306445979be10a91bd4a3febf34c554e is the first bad commit commit bb8f2c32306445979be10a91bd4a3febf34c554e Author: Martin Liska Date: Mon Apr 12 13:42:33 2021 +0200 docs: update symver attribute description caused build failure when configured with: ../gcc/configur

New Swedish PO file for 'gcc' (version 11.1-b20210321)

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 Swedish team of translators. The file is available at: https://translationproject.org/latest/gcc/sv.po (This file, 'gcc-11.1-b20210321.sv.po'

[PATCH] c++: do_class_deduction and dependent init [PR93383]

Here we're crashing during deduction for a template placeholder from a dependent initializer because one of the initializer's elements has an empty TREE_TYPE, something which resolve_args and later unify_one_argument don't expect. And if the deduction from a dependent initializer otherwise fails,

Patch ping for PR95176 fix

Hello, I'd like to ping this patch. It contains two new tree-opt patterns in match.pd. [PATCH] tree-optimization: Optimize division followed by multiply [PR95176] (gnu.org) Thanks, Victor

[PATCH] simplify-rtx: Punt on simplify_{,gen_}subreg to IBM double double if bits are lost [PR99648]

Hi! Similarly to PR95450 done on GIMPLE, this patch punts if we try to simplify_{gen_,}subreg from some constant into the IBM double double IFmode (or sometimes TFmode) if the double double format wouldn't preserve the bits. Not all values are valid in IBM double double, e.g. the format requires

Re: enable __ieee128 for p9vector tests

Hi! Sorry for the late answer. On Fri, Apr 02, 2021 at 01:52:59PM -0300, Alexandre Oliva wrote: > Several compile tests that use the __ieee128 type do not ensure it is > defined. This patch adds -mfloat128 to their command lines, and > disregards the warning that may be issued by it. But they d

Re: [PATCH] combine: Fix up expand_compound_operation [PR99905]

Hi! On Thu, Apr 08, 2021 at 11:24:04AM +0200, Jakub Jelinek wrote: [ snip great explanation ] > The following patch fixes it, by doing the shift in inner_mode properly > and then after the shift doing the lowpart subreg and masking already > in mode. > > Bootstrapped/regtested on x86_64-linux a

Re: [PATCH] c++: Reject alias CTAD in C++17 [PR99008]

On 4/10/21 3:57 PM, Patrick Palka wrote: Here, in C++17 mode, we only pedwarn about the use of alias CTAD and then later ICE from alias_ctad_tweaks when attempting to add a constraint to one of the guides. Since the construction of the guides of an alias template effectively relies on concepts,

Re: [PATCH] c++: variadic class placeholder template deduction [PR97134]

On 4/9/21 5:25 PM, Patrick Palka wrote: do_class_deduction handles specially the case where we're deducing one placeholder from another equivalent one, but here the initializer passed to do_class_deduction is wrapped in an EXPR_PACK_EXPANSION (we're being called from unify during get_partial_spec

Re: [PATCH] c++: constraints are unevaluated operands [PR99961]

On 4/11/21 11:37 AM, Patrick Palka wrote: According to [temp.concept]/6 and [temp.pre]/9, a concept definition and a requires clause are both unevaluated contexts, and hence satisfaction deals only with unevaluated operands, so we should set cp_unevaluated in these three situations. (I guess we

[pushed] c++: premature overload resolution [PR93085]

We can't resolve the call to foo<42> before instantiation of G, because the template parameter of #1 has dependent type. But we were missing that in our dependency check, because the tree walk of DECL_TEMPLATE_PARMS doesn't look into the types of template parameters. So look at them directly. Te

[GCC 12] [PATCH v2] Add inline_ignore_target function attribute

On Mon, Apr 12, 2021 at 2:21 AM Richard Biener wrote: > > On Sat, Apr 10, 2021 at 5:11 PM H.J. Lu via Gcc-patches > wrote: > > > > Add inline_ignore_target function attribute to inform the compiler that > > target specific option mismatch on functions with the always_inline > > attribute may be i

Re: [PATCH] Fix logic error in 32-bit trampolines, PR target/98952

On Fri, 2021-04-09 at 17:09 -0400, Michael Meissner wrote: > Fix logic error in 32-bit trampolines, PR target/98952. > > The test in the PowerPC 32-bit trampoline support is backwards. It aborts > if the trampoline size is greater than the expected size. It should abort > when the trampoline siz

libgo patch committed: Update to Go1.16.3 release

This patch updates libgo to the Go1.16.3 release. Bootstrapped and ran Go testsuite on x86_64-pc-linux-gnu. Committed to mainline. Ian dfbc12e031900c1f30bb94332823d39e5b241b40 diff --git a/gcc/go/gofrontend/MERGE b/gcc/go/gofrontend/MERGE index a3eef239eca..efb7a772882 100644 --- a/gcc/go/gofron

Re: [PATCH] c++: Reject alias CTAD in C++17 [PR99008]

On Mon, 12 Apr 2021, Jason Merrill wrote: > On 4/10/21 3:57 PM, Patrick Palka wrote: > > Here, in C++17 mode, we only pedwarn about the use of alias CTAD and > > then later ICE from alias_ctad_tweaks when attempting to add a > > constraint to one of the guides. Since the construction of the guide

Re: [PATCH] combine: Avoid propagation of (clobber (const_int 0)) into DEBUG_INSNs [PR99830]

Hi! On Fri, Apr 09, 2021 at 09:49:30AM +0200, Jakub Jelinek wrote: > On the following testcase on aarch64 the combiner propagates > (clobber:TI (const_int 0)) into a DEBUG_INSN. Such clobbers are > specific to the combiner, created by gen_lowpart_for_combine: > fail: > return gen_rtx_CLOBBER (

Re: [PATCH] combine: Don't fold away side-effects in simplify_and_const_int_1 [PR99830]

Hi! On Sat, Apr 10, 2021 at 08:02:29AM +0200, Jakub Jelinek wrote: > Here is an alternate patch for the PR99830 bug. > As discussed on IRC and in the PR, the reason why a (clobber:TI (const_int 0)) > has been propagated into the debug insns is that it got optimized away > during simplification fro

Re: [GCC 12] [PATCH v2] Add inline_ignore_target function attribute

On 4/12/21 3:53 PM, H.J. Lu via Gcc-patches wrote: On Mon, Apr 12, 2021 at 2:21 AM Richard Biener wrote: On Sat, Apr 10, 2021 at 5:11 PM H.J. Lu via Gcc-patches wrote: Add inline_ignore_target function attribute to inform the compiler that target specific option mismatch on functions with t

Re: [GCC 12] [PATCH v2] Add inline_ignore_target function attribute

On Mon, Apr 12, 2021 at 4:55 PM Martin Sebor wrote: > > On 4/12/21 3:53 PM, H.J. Lu via Gcc-patches wrote: > > On Mon, Apr 12, 2021 at 2:21 AM Richard Biener > > wrote: > >> > >> On Sat, Apr 10, 2021 at 5:11 PM H.J. Lu via Gcc-patches > >> wrote: > >>> > >>> Add inline_ignore_target function att

[committed] gimple UIDs, LTO and -fanalyzer [PR98599]

On Thu, 2021-01-21 at 22:07 -0500, David Malcolm via Gcc-patches wrote: > On Thu, 2021-01-21 at 20:09 +0100, Jan Hubicka wrote: > > > On Thu, 2021-01-14 at 15:00 +0100, Jan Hubicka wrote: > > > > > On Wed, Jan 13, 2021 at 11:04 PM David Malcolm via > > > > > Gcc-patches > > > > > wrote: > > > > >

Committed: gcc.dg/analyzer/data-model-1.c: Inverse xfail for cris-*-*, PR99212

See PR99212. Now, cris-elf isn't the only target for which this line shows a failure; pru-unknown-elf and m68k-unknown-linux-gnu are two others. I'll leave adjustments to the respective maintainers, but trivially appending more triplets should work: no extra bracketing needed. A specific effectiv

Re: [committed] gimple UIDs, LTO and -fanalyzer [PR98599]

Hello > > Thanks. > > > > I think my earlier analysis was wrong. Sorry for late reply. I was looking into it again yesterday but was bit confused about what is goin gon here. > > > > With the caveat that I'm not as familiar with the IPA code as other > > parts of the compiler, what I think is go

Re: [committed] gimple UIDs, LTO and -fanalyzer [PR98599]

Hi, stepping through the streaming process it turns out to be funny difference between gimple_has_body and node->has_gimple_body_p. While the first tests whether gimple body really exists in memory (by looking for DECL_STRUCT_FUNCTION) the second tests if gimple body can be made available via node-

Re: [PATCH] simplify-rtx: Punt on simplify_{,gen_}subreg to IBM double double if bits are lost [PR99648]

On Mon, 12 Apr 2021, Jakub Jelinek wrote: > Hi! > > Similarly to PR95450 done on GIMPLE, this patch punts if we try to > simplify_{gen_,}subreg from some constant into the IBM double double > IFmode (or sometimes TFmode) if the double double format wouldn't preserve > the bits. Not all values ar