Re: [PATCH] Add debug (slp_tree) and dump infrastructure for this

On Tue, 26 May 2020, David Malcolm wrote: > On Mon, 2020-05-25 at 16:56 +0200, Richard Biener wrote: > > This adds an alternate debug_dump_context similar to the one for > > selftests but for interactive debugging routines. This allows > > to share code between user-visible dumping via the dump_*

Re: [PATCH 0/7] Support vector load/store with length

On Tue, 26 May 2020, Jim Wilson wrote: > On Tue, May 26, 2020 at 12:12 AM Richard Biener wrote: > > From a look at the series description below you seem to add a new way > > of doing loads for this. Did you review other ISAs (those I'm not > > familiar with myself too much are SVE, RISC-V and GC

Re: [PATCH 0/7] Support vector load/store with length

On Tue, 26 May 2020, Segher Boessenkool wrote: > Hi! > > On Tue, May 26, 2020 at 01:29:30PM +0100, Richard Sandiford wrote: > > FWIW, I agree adding .LEN_LOAD and .LEN_STORE seems like a good > > approach. I think it'll be more maintainable in the long run than > > trying to have .MASK_LOADs and

Re: [PATCH 0/7] Support vector load/store with length

Richard Biener writes: > On Tue, 26 May 2020, Jim Wilson wrote: > >> On Tue, May 26, 2020 at 12:12 AM Richard Biener wrote: >> > From a look at the series description below you seem to add a new way >> > of doing loads for this. Did you review other ISAs (those I'm not >> > familiar with myself

[PATCH] Rewrite maintainer-scripts/update_version_git

Hi! This patch rewrites update_version_git to be just a thin wrapper around Martin's new python script. This just arranges to check out the gcc repo in a temporary directory, copy out the contrib scripts so that the running script doesn't change with branch checkouts and runs the script. I've ru

Re: [PATCH] Rewrite maintainer-scripts/update_version_git

On Wed, 27 May 2020, Jakub Jelinek wrote: > Hi! > > This patch rewrites update_version_git to be just a thin wrapper around > Martin's new python script. This just arranges to check out the gcc > repo in a temporary directory, copy out the contrib scripts so that > the running script doesn't cha

Re: [PATCH RFC] gcc-git: Add prepare-commit-msg hook.

On 5/26/20 8:02 PM, Jason Merrill wrote: Maybe these environment variables should start with GCC_GIT, not just GCC, to  give more indication what they're for. All right, I renamed it and pushed to master. Martin

Re: New mklog script

On 5/26/20 8:06 PM, Jason Merrill wrote: gcc-ci-log would be better if you want something short; I'd prefer gcc-commit-mklog, and let people define their own shorter aliases as desired. Hello. There's a rename I'm going to install. Martin >From b423f910dcc2a58a86b61cc5b966a81066abbf12 Mon Se

Re: [PATCH 1/2] rs6000: tune cunroll for simple loops at O2

On Wed, May 27, 2020 at 6:36 AM Jiufu Guo wrote: > > Segher Boessenkool writes: > > > Hi! > > > > On Tue, May 26, 2020 at 08:58:13AM +0200, Richard Biener wrote: > >> On Mon, May 25, 2020 at 7:44 PM Segher Boessenkool > >> wrote: > >> > Yes, cunroll does not have its own option, and that is a pr

Re: New mklog script

On 5/26/20 5:38 PM, Martin Sebor wrote: By the way, it's nice that the existing gcc- aliases are documented on https://gcc.gnu.org/gitwrite.html.  I would suggest to add this one there as well. Yes. I added the documentation bit and pushed ho master. Martin >From 035bdc56110914329c860870e33846

Re: [PATCH 5/7] vect: Support vector load/store with length in vectorizer

Hi Richard, Thanks for your comments! on 2020/5/26 下午8:49, Richard Sandiford wrote: > "Kewen.Lin" writes: >> @@ -626,6 +645,12 @@ public: >>/* True if have decided to use a fully-masked loop. */ >>bool fully_masked_p; >> >> + /* Records whether we still have the option of using a len

[committed] openmp: Fix up omp_declare_variant{s, _alt} htab handling [PR95315]

Hi! This patch fixes a GC ICE. During debugging, I've found that during gimplification we can actually call omp_resolve_declare_variant multiple times and it would create a new magic declare_variant_alt FUNCTION_DECL each time, which is undesirable, once we have such a decl, we should just use th

Re: [PATCH PR95332] gcov-tool: Flexible endian adjustment for merging coverage data

On 5/27/20 5:00 AM, dongjianqiang (A) wrote: Hi GCC maintainers, Proposed patch to PR95332 - gcov-tool merge："not a gcov data file" This error occurs when using gcov-tool merge dir1 dir2 where dir1 and dir2 are the directories containing the .gcda files which were generated by different endia

[PATCH] c++: Try to complete decomp types [PR95328]

Hi! Two years ago Paolo has added the else if (processing_template_decl && !COMPLETE_TYPE_P (type)) pedwarn (...); lines into cp_finish_decomp. For type dependent decl we punt much earlier, but even for types which aren't type dependent COMPLETE_TYPE_P might be false as this testcase shows,

Re: [PATCH 0/7] Support vector load/store with length

on 2020/5/27 下午3:25, Richard Biener wrote: > On Tue, 26 May 2020, Segher Boessenkool wrote: > >> Hi! >> >> On Tue, May 26, 2020 at 01:29:30PM +0100, Richard Sandiford wrote: >>> FWIW, I agree adding .LEN_LOAD and .LEN_STORE seems like a good >>> approach. I think it'll be more maintainable in the

[patch] Add support for __builtin_bswap128

Hi, this patch introduces a new builtin named __builtin_bswap128 on targets where TImode is supported, i.e. 64-bit targets only in practice. The implementation simply reuses the existing double word path in optab, so no routine is added to libgcc (which means that you get two calls to _bswapdi

[PATCH] aarch64: add support for unpacked EOR, ORR and AND

Hi! This patch improves code generation for EOR, ORR and AND on unpacked vectors with SVE. The following function: void f (unsigned int *x, unsigned short *y, unsigned short *z) { for (int i = 0; i < 7; ++i) x[i] = (unsigned short) (y[i] & z[i]); } previously compiled to ptrue p1.d, vl3

[PATCH] Ada: Bump version to 11.

Hello. I'm packaging a new gcc11 package and I noticed this needs to be bumped. Ready for master? Thanks, Martin gcc/ada/ChangeLog: * gnatvsn.ads: Bump Library_Version to 11. --- gcc/ada/gnatvsn.ads | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/gcc/ada/gnatvsn.ad

Re: [PATCH] Ada: Bump version to 11.

> I'm packaging a new gcc11 package and I noticed this needs to be bumped. > > Ready for master? Yes, thanks. > gcc/ada/ChangeLog: > > * gnatvsn.ads: Bump Library_Version to 11. > --- > gcc/ada/gnatvsn.ads | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/gcc/ada/

[PATCH] c++: Fix bogus "does not declare anything" warning (PR 66159)

G++ gives a bogus warning for 'struct A; using B = struct ::A;' complaining that the elaborated-type-specifier doesn't declare anything. That's true, but it's not trying to declare struct ::A, just refer to it unambiguously. Do not emit the warning unless we're actually parsing a declaration. This

Re: drop -aux{dir,base}, revamp -dump{dir,base}

FAIL: outputs exe default 1: a.{out,exe} FAIL: outputs exe default 1: extra a.out FAIL: outputs exe default 2: a.{out,exe} FAIL: outputs exe default 2: extra a.out FAIL: outputs exe savetmp unnamed1: a.{out,exe} FAIL: outputs exe savetmp unnamed1: extra a.out FAIL: outputs exe savetmp unnamed2: a.{

Re: [PATCH 5/7] vect: Support vector load/store with length in vectorizer

"Kewen.Lin" writes: > Hi Richard, > > Thanks for your comments! > > on 2020/5/26 锟斤拷锟斤拷8:49, Richard Sandiford wrote: >> "Kewen.Lin" writes: >>> @@ -626,6 +645,12 @@ public: >>>/* True if have decided to use a fully-masked loop. */ >>>bool fully_masked_p; >>> >>> + /* Records whether

Re: [stage1][PATCH] Make TOPN counter dynamically allocated.

PING^2 On 5/15/20 11:57 AM, Martin Liška wrote: We're in stage1: PING^

Re: drop -aux{dir,base}, revamp -dump{dir,base}

Looks like tcl 8.5.5 has a bug: % glob -nocomplain -path {} -- {a.{out,exe}} % glob -nocomplain -path {} -- {a.{out,exe}*} a.out Andreas. -- Andreas Schwab, SUSE Labs, sch...@suse.de GPG Key fingerprint = 0196 BAD8 1CE9 1970 F4BE 1748 E4D4 88E3 0EEA B9D7 "And now for something completely diffe

Re: [PATCH PR95332] gcov-tool: Flexible endian adjustment for merging coverage data

Hi Martin, Thanks for your comments, I add the ChangeLog in the patch. Thanks, Dong JianQiang On 5/27/20 4:47 PM, Martin Liška wrote: > > On 5/27/20 5:00 AM, dongjianqiang (A) wrote: > > Hi GCC maintainers, > > > > Proposed patch to PR95332 - gcov-tool merge："not a gcov data file" > > > > This

Re: [PATCH PR95332] gcov-tool: Flexible endian adjustment for merging coverage data

On 5/27/20 12:35 PM, dongjianqiang (A) wrote: Thanks for your comments, I add the ChangeLog in the patch. Thanks. That patch is fine, please install it. Martin

Re: [patch] Add support for __builtin_bswap128

On Wed, May 27, 2020 at 11:23:32AM +0200, Eric Botcazou wrote: > gcc/testsuite/ChangeLog: > > * gcc.dg/builtin-bswap-10.c: New test. > * gcc.dg/builtin-bswap-11.c: Likewise. > * gcc.dg/builtin-bswap-12.c: Likewise. > * gcc.target/i386/builtin-bswap-5.c: Likewise. Please us

[PATCH] tree-optimization/95335 - fix SLP nodes dropped to invariant

When we drop a SLP node to invariant because we cannot vectorize it we have to make sure to revisit it in the users. Bootstrapped / tested on x86_64-unknown-linux-gnu, applied. 2020-05-27 Richard Biener PR tree-optimization/95335 * tree-vect-slp.c (vect_slp_analyze_node_operat

Re: [patch] Add support for __builtin_bswap128

> Please use int128 effective target rather than lp64 in the tests that need > __int128 type. OK, thanks, adjusted locally. -- Eric Botcazou

Re: [PATCH v2] arm: Warn if IRQ handler is not compiled with -mgeneral-regs-only [PR target/94743]

Ping? On Thu, 14 May 2020 at 16:57, Christophe Lyon wrote: > > The interrupt attribute does not guarantee that the FP registers are > saved, which can result in problems difficult to debug. > > Saving the FP registers and status registers can be a large penalty, > so it's probably not desirable t

Re: [PATCH] Fix nonconforming memory_operand for vpmov instructions which has memory operand narrow than 128 bits [avx512f]

On Wed, May 27, 2020 at 8:02 AM Hongtao Liu wrote: > > On Mon, May 25, 2020 at 8:41 PM Uros Bizjak wrote: > > > > On Mon, May 25, 2020 at 2:21 PM Hongtao Liu wrote: > > > > > > According to Intel SDM, VPMOVQB xmm1/m16 {k1}{z}, xmm2 has 16-bit > > > memory_operand instead of 128-bit one which e

Re: [PATCH] gcc/Makefile.in: move SELFTEST_DEPS before including language makefile fragments

Hi All, Le 22/05/2020 à 00:13, David Malcolm a écrit : > On Thu, 2020-05-21 at 17:35 +0200, Romain Naour wrote: >> As reported by several Buildroot users [1][2][3], the gcc build >> may fail while running selftests makefile target. >> >> The problem only occurs when ccache is used with gcc 9 and 1

[PATCH] tree-optimization/95356 - fix vectorizable_shift vector types

This makes sure to always use the same vector type for the shift operand as for the shifted operand. Bootstrapped and tested on x86_64-unknown-linux-gnu, applied. Richard. 2020-05-27 Richard Biener PR tree-optimization/95356 * tree-vect-stmts.c (vectorizable_shift): Adjust ve

[committed] libstdc++: Add new testcase for comparison category types

Comparing a comparison category type to anything except a literal 0 is undefined. This verifies that at least some misuses are diagnosed at compile time. * testsuite/18_support/comparisons/categories/zero_neg.cc: New test. Tested x86_64-linux, committed to master. commit 116e3cfc7b8ab8af

Re: [PATCH] Fortran : ICE in gfc_trans_label_assign PR50392

Hi Mark, ping Looks good. Thanks!

[PATCH] tree-optimization/95295 - fix sinking after path merging in new SM code

This fixes a missed sinking of remat stores across unrelated stores after merging from different paths. Bootstrapped and tested on x86_64-unknown-linux-gnu, applied. 2020-05-27 Richard Biener PR tree-optimization/95295 * tree-ssa-loop-im.c (sm_seq_valid_bb): Fix sinking after

RE: [PATCH PR95254] aarch64: gcc generate inefficient code with fixed sve vector length

Hi, > -Original Message- > From: Richard Sandiford [mailto:richard.sandif...@arm.com] > Sent: Tuesday, May 26, 2020 11:58 PM > To: Yangfei (Felix) > Cc: gcc-patches@gcc.gnu.org > Subject: Re: [PATCH PR95254] aarch64: gcc generate inefficient code with > fixed sve vector length > > Sorry

Re: [PATCH] PR94397 the compiler consider "type is( real(kind(1.)) )" as a syntax error

Hi Mark, ping the patch looks good do me. Regards Thomas

Re: [PATCH] PR fortran/95090 - ICE: identifier overflow

Am 26.05.20 um 23:16 schrieb Harald Anlauf: Yet another obvious case of insufficient size of a temporary buffer. OK for master? Yes. Thanks a lot! Regards Thomas

Re: zstd not found if installed in non-system prefix

On 5/20/20 9:32 PM, Michael Kuhn wrote: Hi, when specifying a non-system prefix with --with-zstd, the build fails because the header and library cannot be found (see https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95005). The attached patch fixes the problem and is what we use in Spack to make GCC

[PATCH 1/2] make vect_finish_stmt_generation work w/o stmt_vec_info

This makes the call chain below vec_init_vector happy with a NULL stmt_vec_info which is used as "context". 2020-05-27 Richard Biener * tree-vect-stmts.c (vect_finish_stmt_generation_1): Conditionalize stmt_info use, assert the new stmt cannot throw when not specified.

[PATCH 2/2] Code generate externals/invariants during the SLP graph walk

This reaps the benefit of having the correct vector types on invariant SLP nodes. Bootstrap / regtest of this small series is underway on x86_64-unknown-linux-gnu. Comments welcome. Thanks, Richard. This generates vector defs for externals and invariants during the SLP walk rather than as pa

[PR c++/95263] Revert alias template change

Sadly my attempt to make some aliast template construction immutable doesn't always apply. Reverting that patch and I guess more work needed on modules :( nathan -- Nathan Sidwell 2020-05-27 Nathan Sidwell PR c++/95263, revert 74744bb1f2847b5b9ce3e97e0fec9c23bb0e499f * pt.c (lookup_templ

Re: [patch] Add support for __builtin_bswap128

On Wed, May 27, 2020 at 3:33 PM Eric Botcazou wrote: > > > Please use int128 effective target rather than lp64 in the tests that need > > __int128 type. > > OK, thanks, adjusted locally. OK. Thanks, Richard. > -- > Eric Botcazou

Re: [stage1][PATCH] Lower VEC_COND_EXPR into internal functions.

On 5/26/20 12:15 PM, Richard Sandiford wrote: So longer-term, I think we should replace VCOND(U) with individual ifns, like for VCONDEQ. We could reduce the number of optabs needed by canonicalising greater-based tests to lesser-based tests. Hello. Thanks for the feedback. So would it be poss

Re: [PATCH 0/7] Support vector load/store with length

On Wed, May 27, 2020 at 09:25:43AM +0200, Richard Biener wrote: > On Tue, 26 May 2020, Segher Boessenkool wrote: > > On Tue, May 26, 2020 at 01:29:30PM +0100, Richard Sandiford wrote: > > > FWIW, I agree adding .LEN_LOAD and .LEN_STORE seems like a good > > > approach. I think it'll be more mainta

[pushed] c++: Fix stdcall attribute in template. [PR95222]

Another case that breaks with my fix for PR90750: we shouldn't move type attributes in TYPENAME context either, as there's no decl for them to move to. Tested x86_64-pc-linux-gnu, applying to trunk. gcc/cp/ChangeLog: PR c++/95222 * decl.c (grokdeclarator): Don't shift attributes

[pushed] c++: operator<=> and -Wzero-as-null-pointer-constant [PR95242]

In C++20, if there is no viable operator< available, lhs < rhs gets rewritten to (lhs <=> rhs) < 0, where operator< for the comparison categories is intended to accept literal 0 on the RHS but not other integers. We don't want this to produce a warning from -Wzero-as-null-pointer-constant. Teste

Re: Broken build

On May 26, 2020, Hans-Peter Nilsson wrote: >> Here's a proper patch submission. > And here's an improper bug report. :-) Thanks, H-P, > xgcc: error: : No such file or directory Interesting... If you cut&paste the command line that you included in your so-called improper bug report ;-) do yo

[PATCH] gcc-changelog: enhance handling of renamings

So far, we expect from a commit that renames a file to contain a changelog entry only for the new name. For example, after the following commit: $ git move foo bar $ git commit We expect the following changelog: * bar: Renamed from foo. Git does not keep track of renamings, only file d

Re: drop -aux{dir,base}, revamp -dump{dir,base}

On May 27, 2020, Andreas Schwab wrote: > Looks like tcl 8.5.5 has a bug: Ugh, how unfortunate. > % glob -nocomplain -path {} -- {a.{out,exe}} > % glob -nocomplain -path {} -- {a.{out,exe}*} > a.out Thanks for tracking that down, I'll put in some work around for that. -- Alexandre Oliva, fre

Re: [PATCH, committed] [9/10/11 Regression] PR fortran/95104 - Segfault on a legal WAIT statement

Am 26.05.20 um 23:33 schrieb Harald Anlauf: Committed as obvious. The invalid NULL pointer dereference was discovered by Steve Kargl. Will backport in a few days, when I figure out how to do it now. Thanks for committing this. The way to backport now is to first run contrib/gcc-git-customiza

Re: drop -aux{dir,base}, revamp -dump{dir,base}

On Mai 27 2020, Alexandre Oliva wrote: > On May 27, 2020, Andreas Schwab wrote: > >> Looks like tcl 8.5.5 has a bug: > > Ugh, how unfortunate. In fact, that bug exists in all versions. https://core.tcl-lang.org/tcl/tktview?name=5bbd044812 Andreas. -- Andreas Schwab, SUSE Labs, sch...@suse.de

Re: Broken build

> From: Alexandre Oliva > Date: Wed, 27 May 2020 16:30:07 +0200 > On May 26, 2020, Hans-Peter Nilsson wrote: > > >> Here's a proper patch submission. > > > And here's an improper bug report. > > :-) > > Thanks, H-P, > > > xgcc: error: : No such file or directory > > Interesting... If you

[committed] i386: Implement V2SF shuffles

2020-05-27 Uroš Bizjak gcc/ChangeLog: * config/i386/mmx.md (mmx_pswapdsf2): Add SSE alternatives. Enable insn pattern for TARGET_MMX_WITH_SSE. (*mmx_movshdup): New insn pattern. (*mmx_movsldup): Ditto. (*mmx_movss): Ditto. * config/i386/i386-expand.c (ix86_vectorize_vec_

Re: [PATCH 1/2] make vect_finish_stmt_generation work w/o stmt_vec_info

Richard Biener writes: > This makes the call chain below vec_init_vector happy with a NULL > stmt_vec_info which is used as "context". > > 2020-05-27 Richard Biener > > * tree-vect-stmts.c (vect_finish_stmt_generation_1): > Conditionalize stmt_info use, assert the new stmt cannot th

Re: [PATCH v2] RS6000, add VSX mask manipulation support

GCC maintainers: I have addressed the following comments on the patch from Will: - ChangeLog: fixed name/symbol order; changed reference from rs6000-c.c to rs6000-builtin.def. - define_expand "vec_mtvsrbm": changed name to vec_mtvsrbm_mtvsrbmi, updated comment. - vsx_mask-runnab

Re: [PATCH PR95254] aarch64: gcc generate inefficient code with fixed sve vector length

"Yangfei (Felix)" writes: >> > + >> > +{ >> > + x = x_inner; >> > +} >> > + else if (x_inner != NULL_RTX && MEM_P (y) >> > + && known_eq (GET_MODE_SIZE (x_inner_mode), >> GET_MODE_SIZE (mode)) >> > + && ! targetm.can_change_mode_class (x_inner_mode, mode, >> ALL_REGS) >> > +

Re: [stage1][PATCH] Lower VEC_COND_EXPR into internal functions.

Martin Liška writes: > On 5/26/20 12:15 PM, Richard Sandiford wrote: >> So longer-term, I think we should replace VCOND(U) with individual ifns, >> like for VCONDEQ. We could reduce the number of optabs needed by >> canonicalising greater-based tests to lesser-based tests. > > Hello. > > Thanks f

Re: [PATCH 1/2] make vect_finish_stmt_generation work w/o stmt_vec_info

On May 27, 2020 5:40:30 PM GMT+02:00, Richard Sandiford wrote: >Richard Biener writes: >> This makes the call chain below vec_init_vector happy with a NULL >> stmt_vec_info which is used as "context". >> >> 2020-05-27 Richard Biener >> >> * tree-vect-stmts.c (vect_finish_stmt_generation_

Re: [stage1][PATCH] Lower VEC_COND_EXPR into internal functions.

On May 27, 2020 6:13:24 PM GMT+02:00, Richard Sandiford wrote: >Martin Liška writes: >> On 5/26/20 12:15 PM, Richard Sandiford wrote: >>> So longer-term, I think we should replace VCOND(U) with individual >ifns, >>> like for VCONDEQ. We could reduce the number of optabs needed by >>> canonicali

Re: [PATCH] gcc-changelog: enhance handling of renamings

On 5/27/20 4:29 PM, Pierre-Marie de Rodat wrote: So far, we expect from a commit that renames a file to contain a changelog entry only for the new name. For example, after the following commit: $ git move foo bar $ git commit We expect the following changelog: * bar: Renamed from f

Re: [PATCH] gcc-changelog: enhance handling of renamings

On 5/27/20 7:50 PM, Martin Liška wrote: We'll need here a skip based on version of unidiff. So something like: @pytest.mark.skipif ? I believe something like: diff --git a/contrib/gcc-changelog/git_email.py b/contrib/gcc-changelog/git_email.py index 6e42629cf07..afa6771c7fe 100755 --- a/contr

[PATCH] mklog: support renaming of files

Hi. There's a patch that utilizes newly added functionality in unidiff 0.6.0. It newly generates: diff --git a/gcc/ipa-icf.c b/gcc/ipa-icf2.c similarity index 100% rename from gcc/ipa-icf.c rename to gcc/ipa-icf2.c $ ./contrib/mklog.py 0001-test.patch gcc/ChangeLog: * ipa-icf.c: Moved

[pushed] c++: Handle multiple aggregate overloads [PR95319].

Here, when considering the two 'insert' overloads, we look for aggregate conversions from the same initializer-list to B<3> or initializer_list>. But since my fix for reshape_init overhead on the PR14179 testcase we reshaped the initializer-list directly, leading to an error when we then tried to

[commmitted] jit: fix libgccjit.info entry [PR 91330]

2020-05-27 Tom Tromey gcc/jit/ChangeLog: PR jit/91330 * docs/conf.py (texinfo_documents): Set description. * docs/_build/texinfo/libgccjit.texi: Regenerate. --- gcc/jit/docs/_build/texinfo/libgccjit.texi | 2 +- gcc/jit/docs/conf.py | 2 +- 2 files

[committed] jit: use deep unsharing of trees [PR 95314]

PR jit/95314 reports a internal error inside verify_gimple, which turned out to be due to reusing the result of gcc_jit_lvalue_get_address in several functions, leading to tree nodes shared between multiple function bodies. This patch fixes the issue by adopting the "Deep unsharing" strategy descr

[committed] i386: Remove %q modifier from two pmov insn templates [PR95355]

2020-05-27 Uroš Bizjak gcc/ChangeLog: PR target/95355 * config/i386/sse.md (avx512f_v16qiv16si2): Remove %q operand modifier from insn template. (avx512f_v8hiv8di2): Ditto. gcc/testsuite/ChangeLog: PR target/95355 * gcc.target/i386/pr95355.c: New test. Bootstrapped

Aw: Re: [PATCH, committed] [9/10/11 Regression] PR fortran/95104 - Segfault on a legal WAIT statement

Hi Thomas, thanks for the hint: > Von: "Thomas Koenig" > Am 26.05.20 um 23:33 schrieb Harald Anlauf: > > Will backport in a few days, when I figure out how to do it now. > > The way to backport now is to first run contrib/gcc-git-customization.sh > from current master, and then change to the bra

[comitted] i386: Fix V2SF horizontal add/subtract insns

PFPNACC insn is incorrectly modelled to perform addition and subtraction of two operands, but in reality it performs horizontal addition and subtraction: Instruction: PFPNACC dest,src Description: dest[31:0] <- dest[31:0] - dest[63:32]; dest[63:32] <- src[31:0] + src[63:32]; Also, it is not poss

[PATCH] c++: constexpr RANGE_EXPR ctor indexes [PR95241]

In the testcase below, the CONSTRUCTOR for 'field' contains a RANGE_EXPR index: {aggr_init_expr<...>, [1...2]={.off=1}} but get_or_insert_ctor_field isn't prepared to handle RANGE_EXPR indexes. This patch adds limited support for RANGE_EXPR indexes to get_or_insert_ctor_field. The limited sco

Re: [PATCH] c++: constexpr RANGE_EXPR ctor indexes [PR95241]

On Wed, 27 May 2020, Patrick Palka wrote: > In the testcase below, the CONSTRUCTOR for 'field' contains a > RANGE_EXPR index: > > {aggr_init_expr<...>, [1...2]={.off=1}} > > but get_or_insert_ctor_field isn't prepared to handle RANGE_EXPR > indexes. > > This patch adds limited support for RAN

[committed] libstdc++: Fix std::reverse_iterator comparisons (PR 94354)

The std::reverse_iterator comparisons have always been implemented only in terms of equality and less than. In C++98 that made no difference for reasonable code, because when the underlying operators are the same type they are required to support all comparisons anyway. But since LWG 280 it's poss

[committed] libstdc++: Fix view adaptors for mixed-const sentinels and iterators (PR 95322)

The bug report is that transform_view's sentinel cannot be compared to its iterator. The comparison is supposed to use operator==(iterator, sentinel) after converting sentinel to sentinel. However, the operator== is a hidden friend so is not a candidate when comparing iterator with sentinel. The r

Re: [PATCH] c++: constexpr RANGE_EXPR ctor indexes [PR95241]

On Wed, 27 May 2020, Patrick Palka wrote: > On Wed, 27 May 2020, Patrick Palka wrote: > > > In the testcase below, the CONSTRUCTOR for 'field' contains a > > RANGE_EXPR index: > > > > {aggr_init_expr<...>, [1...2]={.off=1}} > > > > but get_or_insert_ctor_field isn't prepared to handle RANGE_E

Re: drop -aux{dir,base}, revamp -dump{dir,base}

outputs.exp: no lto, linker default output, cdtor temps, empty args From: Alexandre Oliva This patch fixes various issues in the testsuite that came up after the dump/aux output revamp, namely: - many outputs.exp tests used -flto without checking that LTO was supported, getting lots of failures

[committed] libstdc++: Fix atomic::load (PR 95282)

PR libstdc++/95282 * include/bits/atomic_base.h (__atomic_impl::load): Add cv-qualifiers to parameter so that _Tp is deduced as the unqualified type. * testsuite/29_atomics/atomic_float/95282.cc: New test. Tested powerpc64le-linbux, committed to master. Bac

Re: drop -aux{dir,base}, revamp -dump{dir,base}

On Wed, 2020-05-27 at 19:05 -0300, Alexandre Oliva wrote: > outputs.exp: no lto, linker default output, cdtor temps, empty args > > From: Alexandre Oliva > > This patch fixes various issues in the testsuite that came up after > the dump/aux output revamp, namely: > > - many outputs.exp tests us

Re: [PATCH] gcc: xtensa: delegitimize UNSPEC_PLT

On Tue, May 26, 2020 at 11:43 AM Max Filippov wrote: > > This fixes 'non-delegitimized UNSPEC 3 found in variable location' notes > issued when building libraries which interferes with running tests. > > 2020-05-24 Max Filippov > gcc/ > * config/xtensa/xtensa.c (xtensa_delegitimize_addr

Re: Broken build

Hans-Peter Nilsson via Gcc-patches writes: > And here's an improper bug report. > > One of the commits between cfdff3eeb90..5c8344e7289 caused every > single *linked* test to fail for cris-elf, like: I can confirm that the moxie-elf test cases don't link either. It looks like setting ldscript i

Re: Broken build

On May 27, 2020, Hans-Peter Nilsson wrote: >> I ask because this error suggests an empty argument passed to >> GCC. > And ignored before your rewrite? Or absent. It turned out my massaging of ldflags et al turned consecutive blanks into empty arguments. I posted a patch for that and most of t

Re: [PATCH 5/7] vect: Support vector load/store with length in vectorizer

on 2020/5/27 下午6:02, Richard Sandiford wrote: > "Kewen.Lin" writes: >> Hi Richard, >> >> Thanks for your comments! >> >> on 2020/5/26 锟斤拷锟斤拷8:49, Richard Sandiford wrote: >>> "Kewen.Lin" writes: @@ -626,6 +645,12 @@ public: /* True if have decided to use a fully-masked loop. */

Re: [PATCH] Port libgccjit to Windows.

Hi, > Do you have commit/push access to the gcc repository? No I don't. > BTW, why isn't it necessary to use --enable-host-shared in Windows? > Can we document that? That's because all code is position independent in Windows. > On the subject of nitpicking, I find myself getting distracted by

Re: [PATCH] gcc: xtensa: delegitimize UNSPEC_PLT

On Wed, May 27, 2020 at 4:35 PM augustine.sterl...@gmail.com wrote: > > On Tue, May 26, 2020 at 11:43 AM Max Filippov wrote: > > > > This fixes 'non-delegitimized UNSPEC 3 found in variable location' notes > > issued when building libraries which interferes with running tests. > > > > 2020-05-24

Re: [PATCH PR95332] gcov-tool: Flexible endian adjustment for merging coverage data

Thanks for reviewing this. Could you please help install this patch? I am not a gcc commiter. Regards, Dong JianQiang > > On 5/27/20 12:35 PM, dongjianqiang (A) wrote: > > Thanks for your comments, I add the ChangeLog in the patch. > > Thanks. That patch is fine, please install it. > > Marti

Ping^1 [PATCH 2/4 V3] Add target hook stride_dform_valid_p

Hi, Gentle ping patches as below: 1/4 v3 https://gcc.gnu.org/pipermail/gcc-patches/2020-February/540171.html 2/4 v3 https://gcc.gnu.org/pipermail/gcc-patches/2020-March/541387.html 3/4 v3 https://gcc.gnu.org/pipermail/gcc-patches/2020-May/545643.html Or shall I ping them seperately? Thanks! Kew

Re: [PATCH] Fix nonconforming memory_operand for vpmov instructions which has memory operand narrow than 128 bits [avx512f]

On Wed, May 27, 2020 at 8:01 PM Uros Bizjak wrote: > > On Wed, May 27, 2020 at 8:02 AM Hongtao Liu wrote: > > > > On Mon, May 25, 2020 at 8:41 PM Uros Bizjak wrote: > > > > > > On Mon, May 25, 2020 at 2:21 PM Hongtao Liu wrote: > > > > > > > > According to Intel SDM, VPMOVQB xmm1/m16 {k1}{z},

Re: [PATCH PR95332] gcov-tool: Flexible endian adjustment for merging coverage data

On 5/28/20 4:07 AM, dongjianqiang (A) wrote: Thanks for reviewing this. Could you please help install this patch? I am not a gcc commiter. I've just done that. For the next time, please add ChangeLog entries to a git commit message. We do not longer modify ChangeLog filese, these are generat

Re: [PATCH] Fix nonconforming memory_operand for vpmov instructions which has memory operand narrow than 128 bits [avx512f]

On Thu, May 28, 2020 at 7:10 AM Hongtao Liu wrote: > > On Wed, May 27, 2020 at 8:01 PM Uros Bizjak wrote: > > > > On Wed, May 27, 2020 at 8:02 AM Hongtao Liu wrote: > > > > > > On Mon, May 25, 2020 at 8:41 PM Uros Bizjak wrote: > > > > > > > > On Mon, May 25, 2020 at 2:21 PM Hongtao Liu wrote: