Re: [PATCH 1/4] unroll: Add middle-end unroll factor estimation

2021-02-28 Thread Kewen.Lin via Gcc-patches
on 2021/1/27 下午5:43, Kewen.Lin via Gcc-patches wrote: > on 2021/1/26 下午6:53, Richard Biener wrote: >> On Tue, 26 Jan 2021, Kewen.Lin wrote: >> >>> Hi Segher/Richard B./Richard S., >>> >>> Many thanks for your all helps and comments on this! >>> >>> on 2021/1/25 下午3:56, Richard Biener wrote:

Re: [PATCH 1/5] std::latch: reduce internal implementation from ptrdiff_t to int

2021-02-28 Thread Hans-Peter Nilsson
On Fri, 26 Feb 2021, Thiago Macieira via Gcc-patches wrote: > On Friday, 26 February 2021 11:31:00 PST Andreas Schwab wrote: > > On Feb 26 2021, Thiago Macieira wrote: > > > On Friday, 26 February 2021 10:14:42 PST Andreas Schwab wrote: > > >> On Feb 26 2021, Thiago Macieira via Gcc-patches

[PATCH 6/6] c++: Consolidate REQUIRES_EXPR evaluation/diagnostic routines

2021-02-28 Thread Patrick Palka via Gcc-patches
This folds the diagnose_requires_expr routines into the corresponding tsubst_requires_expr ones. This is achieved by making the latter routines take a sat_info instead of a subst_info, and assigning the appropriate meanings to the flags sat_info::noisy and sat_info::diagnose_unsatisfaction_p

[PATCH 5/6] c++: Clean up normalization / satisfaction routines

2021-02-28 Thread Patrick Palka via Gcc-patches
This patch mostly performs some straightforward refactoring: - Renamed satisfy_constraint to satisfy_normalized_constraints - Renamed the three-parameter version of satisfy_constraint_expression to satisfy_nondeclaration_constraints - Removed normalize_(non)?template_requirements -

Re: [PATCH 4/4] c++: dependent constraint on placeholder 'auto' [PR96443]

2021-02-28 Thread Patrick Palka via Gcc-patches
On Fri, 12 Feb 2021, Jason Merrill wrote: > On 2/11/21 5:14 PM, Patrick Palka wrote: > > On Thu, 11 Feb 2021, Jason Merrill wrote: > > > > > On 2/8/21 2:03 PM, Patrick Palka wrote: > > > > This fixes the way we check satisfaction of constraints on placeholder > > > > types in various contexts,

Re: [x86] Fix PR target/99264

2021-02-28 Thread Uros Bizjak via Gcc-patches
On Fri, Feb 26, 2021 at 5:14 PM Eric Botcazou wrote: > > Hi, > > this wrong-code PR for the C++ compiler on x86-64/Windows is a regression > in GCC 9 and later, but the underlying issue has probably been there since > SEH was implemented and is exposed by this comment in config/i386/winnt.c: > >

Re: [PATCH 3/4] c++: Delay normalizing nested requirements until satisfaction

2021-02-28 Thread Patrick Palka via Gcc-patches
On Fri, 12 Feb 2021, Jason Merrill wrote: > On 2/10/21 9:41 AM, Patrick Palka wrote: > > On Tue, 9 Feb 2021, Jason Merrill wrote: > > > > > On 2/8/21 2:03 PM, Patrick Palka wrote: > > > > This sets up the functionality for controlling the initial set of > > > > template parameters to pass to

[pushed] testsuite, coroutines : Mark final awaiters and co_await operators noexcept.

2021-02-28 Thread Iain Sandoe
Hi, Fixing up some non-conforming testcases. This is part of the requirement of [dcl.fct.def.coroutine]/15. In addition to promise final_suspend() calls, the following cases must also be noexcept as per discussion in PR95616. - finalSuspendObj.operator co_await() -

Re: [PATCH 4/5] barrier: use int instead of unsigned char for the phase state

2021-02-28 Thread Hans-Peter Nilsson
On Fri, 26 Feb 2021, Thiago Macieira via Gcc-patches wrote: > ints can be used in futexes. chars can't. Shouldn't that be an atomic type instead of a bare int then? > --- > libstdc++-v3/include/std/barrier | 21 - > 1 file changed, 12 insertions(+), 9 deletions(-) > > diff

Re: [PATCH] coroutines : Remove throwing_cleanup marks from the ramp [PR95822].

2021-02-28 Thread Iain Sandoe
Jason Merrill wrote: On 2/26/21 4:36 PM, Iain Sandoe wrote: Jason Merrill wrote: On 2/24/21 3:06 PM, Iain Sandoe wrote: So (a) the 'throwing_cleanup' is no longer correct for the ramp and (b) we do not need to transfer it to the actor which only contains void returns. OK,

[PATCH] Add missing changes to Makefile.tpl

2021-02-28 Thread H.J. Lu via Gcc-patches
On Sat, Feb 27, 2021 at 11:01 PM Mike Frysinger wrote: > > On 19 Dec 2020 10:10, H.J. Lu via Gdb-patches wrote: > > --- a/Makefile.in > > +++ b/Makefile.in > > > > +PGO_BUILD_TRAINING_FLAGS_TO_PASS = \ > > + PGO_BUILD_TRAINING=yes \ > > + CFLAGS_FOR_TARGET="$(PGO_BUILD_TRAINING_CFLAGS)" \

[r11-7430 Regression] FAIL: g++.dg/ext/attr-lambda3.C -std=c++2a (test for excess errors) on Linux/x86_64

2021-02-28 Thread sunil.k.pandey via Gcc-patches
On Linux/x86_64, 5d9d6c1cd8d9f0e057b4a7a849bc765e2109137c is the first bad commit commit 5d9d6c1cd8d9f0e057b4a7a849bc765e2109137c Author: Jason Merrill Date: Fri Feb 26 05:45:02 2021 -0500 c++: Allow GNU attributes before lambda -> [PR90333] caused FAIL: g++.dg/ext/attr-lambda3.C

[committed] fortran: Fix up wording of 3 OpenMP diagnostic messages [PR99303]dbI

2021-02-28 Thread Jakub Jelinek via Gcc-patches
Hi! As reported in the PR, some diagnostic messages need wording improvements. Bootstrapped/regtested on x86_64-linux and i686-linux, committed to trunk. 2021-02-28 Jakub Jelinek PR fortran/99303 * openmp.c (gfc_omp_requires_add_clause): Fix up diagnostic message

[committed] fortran: Replace double space in diagnostic with a single one [PR99300]

2021-02-28 Thread Jakub Jelinek via Gcc-patches
Hi! The diagnostic had " " in it instead of just " ". Bootstrapped/regtested on x86_64-linux and i686-linux, committed to trunk as obvious. 2021-02-28 Jakub Jelinek PR fortran/99300 * frontend-passes.c (doloop_code): Replace double space in diagnostics with a single

[commit] typos: Fix various typos - mainly misspelled reference* [PR99304]

2021-02-28 Thread Jakub Jelinek via Gcc-patches
Hi! The PR is about a typo in handle_malloc_attribute diagnostic message, but grepping around I found many other cases and while fixing those I've noticed a couple of other typos. Bootstrapped/regtested on x86_64-linux and i686-linux, committed to trunk. 2021-02-28 Jakub Jelinek PR

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

2021-02-28 Thread Translation Project Robot
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,