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: Sunday, May 31, 2020 12:01 AM > To: Yangfei (Felix) > Cc: gcc-patches@gcc.gnu.org; Uros Bizjak ; Jakub > Jelinek ; Hongtao Liu ; H.J. Lu > > Subject: Re: [PATCH PR95254] aarch64: gcc generate

[PATCH] [stage1] ipa-cp: Fix PGO regression caused by r278808

resend the patch for stage1: https://gcc.gnu.org/pipermail/gcc-patches/2020-January/538186.html The performance of exchange2 built with PGO will decrease ~28% by r278808 due to profile count set incorrectly. The cloned nodes are updated to a very small count caused later pass cunroll fail to

[committed] wwwdocs: Update link for ARC.

Pushed. --- htdocs/readings.html | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/htdocs/readings.html b/htdocs/readings.html index 3d654a37..2488ca9d 100644 --- a/htdocs/readings.html +++ b/htdocs/readings.html @@ -78,7 +78,7 @@ names. arc Manufacturer: Synopsys Inc (as

[committed] libstdc++: Switch www.cs.princeton.edu to https

My first libstdc+++ commit in the new (ChangeLog) new (Git) world order, so particularly happy for advise on any mistakes or potential improvements. (Apart from the stray change that sneaked into the .xml file in the last minute somehow :-( -- that one's resolved already.) Pushed. Gerald

[committed] wwwdocs: Remove duplicate (old) Blackfin reference.

Pushed. Gerald --- htdocs/readings.html | 1 - 1 file changed, 1 deletion(-) diff --git a/htdocs/readings.html b/htdocs/readings.html index 09420335..3d654a37 100644 --- a/htdocs/readings.html +++ b/htdocs/readings.html @@ -94,7 +94,6 @@ names. Blackfin Manufacturer: Analog Devices -

[committed] wwwdocs: Update reference to Intel's pcommit deprecation.

Pushed. Gerald --- htdocs/gcc-5/changes.html | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/htdocs/gcc-5/changes.html b/htdocs/gcc-5/changes.html index 536ab6c0..efa322b1 100644 --- a/htdocs/gcc-5/changes.html +++ b/htdocs/gcc-5/changes.html @@ -1084,7 +1084,7 @@ are not

[PATCH 0/2] x86: Add cmpmemsi for -minline-all-stringops

We used to expand memcmp to "repz cmpsb" via cmpstrnsi.  It was changed by commit 9b0f6f5e511ca512e4faeabc81d2fd3abad9b02f Author: Nick Clifton Date: Fri Aug 12 16:26:11 2011 + builtins.c (expand_builtin_memcmp): Do not use cmpstrnsi pattern. * builtins.c

[PATCH 2/2] x86: Add cmpmemsi for -minline-all-stringops

We used to expand memcmp to "repz cmpsb" via cmpstrnsi.  It was changed by commit 9b0f6f5e511ca512e4faeabc81d2fd3abad9b02f Author: Nick Clifton Date: Fri Aug 12 16:26:11 2011 + builtins.c (expand_builtin_memcmp): Do not use cmpstrnsi pattern. * builtins.c

[PATCH 1/2] x86: Pass a copy of the string length to cmpstrnqi

cmpstrnsi expander may pass the actual string length directly to cmpstrnqi patterns. For cmpstrnsi, one of the strings must be a constant and expand_builtin_strncmp rewrites the length argument to be the minimum of the const string length and the actual string length. But it is not the case for

[PATCH] Fix unrecognised -mcpu target: armv7-a on arm-wrs-vxworks7 (PR95420)

Hi, In the removal of arm-wrs-vxworks, the default cpu was updated from arm8 to armv7-a, but this is not recognized as a valid -mcpu target. There is however generic-armv7-a, which was likely the intended cpu that should have been used instead. Tested by building a cross-compiler targetting

[committed] wwwdocs: Simplify language in gitwrite.html.

I noticed we have a tendency of using overly long language in our process documentation. This is a first step a simplifying (and trimming). Pushed. Gerald --- htdocs/gitwrite.html | 24 ++-- 1 file changed, 10 insertions(+), 14 deletions(-) diff --git

Re: [PATCH] diagnostics: Add function call parens matching to c_parser.

On Sun, May 24, 2020 at 11:46:34PM +0200, Mark Wielaard wrote: > The C++ parser already tracks function call parens matching, but the C > parser doesn't. This adds the same functionality to the C parser and adds > a testcase showing the C++ and C parser matching function call parens > in an error

Re: [PATCH 4/6] contrib: Add or1k-elf, or1k-linux-*, and or1k-rtems to config-list.mk

On 31/05/2020 22:53, Stafford Horne wrote: > On Sun, May 31, 2020 at 12:19:16PM +0200, Iain Buclaw wrote: >> Support for OpenRISC target was added in SVN r265963. >> >> The target configurations were taken from the list of supported >> toolchains[1], so seems sensible to include them all. >> >>

Re: [PATCH 4/6] contrib: Add or1k-elf, or1k-linux-*, and or1k-rtems to config-list.mk

On Sun, May 31, 2020 at 12:19:16PM +0200, Iain Buclaw wrote: > Support for OpenRISC target was added in SVN r265963. > > The target configurations were taken from the list of supported > toolchains[1], so seems sensible to include them all. > > OK? > > Regards > Iain > > [1]:

[pushed] coroutines: Avoid functions with unlowered coroutine trees [PR95087].

Hi, Diagnosing bad uses of 'return' in coroutines is somewhat tricky, since the user can use the keyword before we know that the function is a coroutine (where such returns are not permitted). At present, we are just doing a check for any use of 'return' and erroring on that. However, we can't

[pushed] coroutines: Remove up some unused values.

Hi, The build_new_method_call allows us to inspect the function decl used. In most cases, this is not used and effectively is a set but not used value. We can just set the parm to NULL. tested on x86_64-darwin, linux, powerpc64-linux pushed to master, thanks Iain gcc/cp/ChangeLog: *

Fwd: [patch] substitute_and_fold_engine merge with evrp domwalker

PING -- Forwarded message - From: Aldy Hernandez Date: Mon, May 18, 2020 at 7:59 PM Subject: [patch] substitute_and_fold_engine merge with evrp domwalker To: Jeff Law Cc: gcc-patches Howdy. The main evrp domwalker seems cut and pasted from the substitute_and_fold_engine (or

Re: [patch] move array bounds checking into its own file

On Sun, May 31, 2020 at 7:31 PM Jeff Law wrote: > > On Mon, 2020-05-18 at 20:11 +0200, Aldy Hernandez wrote: > > As a follow-up to the patch moving array bounds checking into its own > > class, this moves the class into its own files. As I've mentioned > > previously, having it in tree-vrp just

Re: [committed] Fix latent bug due to peephole2 pass dropping REG_INC notes

On Mon, 2020-06-01 at 02:28 +0900, Oleg Endo wrote: > Hi Jeff, > > On Sun, 2020-05-31 at 11:20 -0600, Jeff Law via Gcc-patches wrote: > > The peephole2 pass makes some attempt to update various notes, but that > > doesn't > > include REG_INC notes. While I could trivially fix this in the H8

Re: [patch] move value_range_equiv class to own file

On Mon, 2020-05-18 at 20:08 +0200, Aldy Hernandez wrote: > We already moved the value_range class into its own class in the last > release. I think it's time to move the value_range_equiv stuff into its > own class, as it's a distraction from the VRP code. > > I've moved it to

Re: [patch] move array bounds checking into its own file

On Mon, 2020-05-18 at 20:11 +0200, Aldy Hernandez wrote: > As a follow-up to the patch moving array bounds checking into its own > class, this moves the class into its own files. As I've mentioned > previously, having it in tree-vrp just pollutes the file with unrelated > stuff. > > Jeff, I

Re: [committed] Fix latent bug due to peephole2 pass dropping REG_INC notes

Hi Jeff, On Sun, 2020-05-31 at 11:20 -0600, Jeff Law via Gcc-patches wrote: > > The peephole2 pass makes some attempt to update various notes, but that > doesn't > include REG_INC notes. While I could trivially fix this in the H8 port, I > wouldn't be terribly surprised if the lack of a

Re: [PATCH 5/6] contrib: Add v850e1-elf to config-list.mk

On Sun, 2020-05-31 at 12:19 +0200, Iain Buclaw via Gcc-patches wrote: > This comment was added in SVN r173410, v850e1-* was added to config.sub > in SVN r174691i (around 2011). So it should no longer apply. > > OK? > > Regards > Iain > > --- > contrib/ChangeLog: > > * config-list.mk

Re: [PATCH 4/6] contrib: Add or1k-elf, or1k-linux-*, and or1k-rtems to config-list.mk

On Sun, 2020-05-31 at 12:19 +0200, Iain Buclaw via Gcc-patches wrote: > Support for OpenRISC target was added in SVN r265963. > > The target configurations were taken from the list of supported > toolchains[1], so seems sensible to include them all. > > OK? > > Regards > Iain > > [1]:

Re: [PATCH 1/6] contrib: Remove cris-linux and crisv32-* from config-list.mk

On Sun, 2020-05-31 at 12:02 +0200, Iain Buclaw via Gcc-patches wrote: > Hi, > > Continuing from the previous update to config-list.mk, I realize that > there are a few other more additions/removals to be done. > > To start off, support for crisv32-*-* and cris-*-linux* was removed in > git

Re: [PATCH 2/6] contrib: Remove arm-wrs-vxworks from config-list.mk

On Sun, 2020-05-31 at 12:15 +0200, Iain Buclaw via Gcc-patches wrote: > Support for arm-wrs-vxworks was removed in git 27204060db5/r10-4684. > > Looking at the commit, it seems that it can instead be replaced with > arm-wrs-vxworks7, however this target doesn't pass selftests due to an >

[committed] Fix latent bug due to peephole2 pass dropping REG_INC notes

The H8 recently started regressing 20071219-1.c on the H8/SX with -mint32. I didn't really dig into what change caused the regression. While I recently changed this peephole, it was just collapsing 3 patterns into 1 using mode iterators. So more likely something earlier in the pipeline just

Cleanup indexable tree ref streaming

Hi, this patch removes some abstraction around streaming references which makes it difficult to play with the format of actual streamed references. They are alway spair of integers, one represents the tag and other is index. So I separated code computing them to common place followed by the uhwi

Re: [PATCH 3/6] contrib: Add pru-elf to config-list.mk

On неделя, 31 май 2020 г. 13:17:21 EEST Iain Buclaw wrote: > Support for the TI PRU target was added in SVN r272202. > > Judging from the testsuite results posted at the time[1], the only > supported target is pru-elf. > > OK? > > Regards > Iain. > > [1]:

[committed] wwwdocs: Update examples for branches and tags to newer ones.

Pushed. --- htdocs/git.html | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/htdocs/git.html b/htdocs/git.html index bec93ead..8c28bc02 100644 --- a/htdocs/git.html +++ b/htdocs/git.html @@ -146,12 +146,12 @@ series, Y is always nonzero and Z is always zero for a

[PATCH] tilepro: Update generator file to define IN_TARGET_CODE in target file.

Hi, The target files tilegx/mul-tables.c and tilepri/mul-tables.c were updated in SVN r255743, but the generator file that produces them wasn't, so it was reverting this change during builds. Only tested by running make all-gcc for all tile*-*-* targets present in config-list.mk. OK? Regards

[PATCH 6/6] contrib: Add OPT-enable-obsolete to tile*-*-*

The tile*-*-* targets were marked as obsolete in SVN r259724. OK? Regards Iain --- contrib/ChangeLog: * config-list.mk (LIST): Add OPT-enable-obsolete to tilegx-linux-gnu, tilegxbe-linux-gnu, and tilepro-linux-gnu. --- contrib/config-list.mk | 3 ++- 1 file changed, 2

[PATCH 5/6] contrib: Add v850e1-elf to config-list.mk

This comment was added in SVN r173410, v850e1-* was added to config.sub in SVN r174691i (around 2011). So it should no longer apply. OK? Regards Iain --- contrib/ChangeLog: * config-list.mk (LIST): Add v850e1-elf. --- contrib/config-list.mk | 3 +-- 1 file changed, 1 insertion(+), 2

[PATCH 4/6] contrib: Add or1k-elf, or1k-linux-*, and or1k-rtems to config-list.mk

Support for OpenRISC target was added in SVN r265963. The target configurations were taken from the list of supported toolchains[1], so seems sensible to include them all. OK? Regards Iain [1]: https://www.openrisc.io/software --- contrib/ChangeLog: * config-list.mk (LIST): Add

[PATCH 3/6] contrib: Add pru-elf to config-list.mk

Support for the TI PRU target was added in SVN r272202. Judging from the testsuite results posted at the time[1], the only supported target is pru-elf. OK? Regards Iain. [1]: http://dinux.eu/gnupru/testresults/index.html --- contrib/ChangeLog: * config-list.mk (LIST): Add pru-elf.

[PATCH 2/6] contrib: Remove arm-wrs-vxworks from config-list.mk

Support for arm-wrs-vxworks was removed in git 27204060db5/r10-4684. Looking at the commit, it seems that it can instead be replaced with arm-wrs-vxworks7, however this target doesn't pass selftests due to an unrecognized CPU (PR 95420). Nor does the previous default CPU work either

[PATCH 1/6] contrib: Remove cris-linux and crisv32-* from config-list.mk

Hi, Continuing from the previous update to config-list.mk, I realize that there are a few other more additions/removals to be done. To start off, support for crisv32-*-* and cris-*-linux* was removed in git 2b36e4dc813/r11-214. OK? Regards Iain --- contrib/ChangeLog: * config-list.mk