Re: how to find variable related to a virtual ssa name

2020-05-13 Thread Richard Biener via Gcc
On Wed, May 13, 2020 at 6:03 AM 易会战 wrote: > > It seems the function ptr_deref_may_alias_global_p cannot give right result. > For example, > int func(int size, int i) > { > int * sum; > sum = malloc() > here some code access sum pointing to memory > return sum[i] > } >

Re: size of exception handling (Was: performance of exception handling)

2020-05-13 Thread David Brown
On 13/05/2020 00:48, Jonathan Wakely via Gcc wrote: On Tue, 12 May 2020 at 23:39, Jonathan Wakely wrote: On Tue, 12 May 2020, 21:57 Freddie Chopin, wrote: Anyway... If you have to recompile the toolchain, the problem is still there. Most of the people (like 99,666%) will not do that for

Re: Automatically generated ChangeLog files - PHASE 1

2020-05-13 Thread Richard Sandiford
Martin Liška writes: > Hi. > > Thanks to Jakub, we finally set up an experimental environment: > gcc.gnu.org/home/gccadmin/gcc-reposurgeon-8.git > > The repository now contains a new pre-commit hook that validates > the git commit format ([1]) and provides a reasonable error message > when

Re: Automatically generated ChangeLog files - PHASE 1

2020-05-13 Thread Jozef Lawrynowicz
Hi, On Tue, 12 May 2020 11:05:58 +0200 Martin Liška wrote: > Hi. > > Thanks to Jakub, we finally set up an experimental environment: > gcc.gnu.org/home/gccadmin/gcc-reposurgeon-8.git > > The repository now contains a new pre-commit hook that validates > the git commit format ([1]) and

Re: Automatically generated ChangeLog files - PHASE 1

2020-05-13 Thread Thomas Koenig via Gcc
Hi Martin, Thanks to Jakub, we finally set up an experimental environment: gcc.gnu.org/home/gccadmin/gcc-reposurgeon-8.git How does one go about testing this? It would be helpful if you could explain this assuming that the Fortran people have at best an extremely shallow knowledge of git and

Re: [RFC] Closing of all remaining Bugzilla PRs against powerpcspe

2020-05-13 Thread Segher Boessenkool
On Wed, May 13, 2020 at 01:42:37AM +0100, Maciej W. Rozycki wrote: > On Sat, 9 May 2020, Eric Botcazou wrote: > > > > Strangely, I failed to find any PR for e200, so maybe some unnoticed ones > > > are still lying around. > > > > I think that the e200 support was never contributed upstream. > >

Re: [RFC] Closing of all remaining Bugzilla PRs against powerpcspe

2020-05-13 Thread Maciej W. Rozycki
On Wed, 13 May 2020, Segher Boessenkool wrote: > > > I think that the e200 support was never contributed upstream. > > > > Or rather, it wasn't accepted. Cf. > > , > >

ChangeLog files - server and client scripts (git cherry-pick)

2020-05-13 Thread Martin Liška
On 5/13/20 1:05 PM, Martin Liška wrote: I suggest to use native 'git revert XYZ' and 'git cherry-pick -x XYZ'. I've prepared a working version of Revert format: https://github.com/marxin/gcc-changelog/tree/cherry-pick So using git cherry-pick -x HASH one gets something like: $ cat

Re: ChangeLog files - server and client scripts

2020-05-13 Thread Martin Liška
The scripts were just installed to master except the git alias. I'm sending that in a separate patch. Now the alias can be used from any subfolder in a gcc git repository. Martin >From eb47191e8d8cbbda285c4df7eb2d1e98091edab9 Mon Sep 17 00:00:00 2001 From: Martin Liska Date: Wed, 13 May 2020

dejagnu version update?

2020-05-13 Thread Mike Stump via Gcc
I've changed the subject to match the 2015, 2017 and 2018 email threads. On May 13, 2020, at 3:26 AM, Thomas Schwinge wrote: > > Comparing DejaGnu/GCC testsuite '*.sum' files between two systems ("old" > vs. "new") that ought to return identical results, I found that they > didn't: > I have

Re: dejagnu version update?

2020-05-13 Thread Jonathan Wakely via Gcc
On Wed, 13 May 2020 at 18:19, Mike Stump via Gcc wrote: > > I've changed the subject to match the 2015, 2017 and 2018 email threads. > > On May 13, 2020, at 3:26 AM, Thomas Schwinge wrote: > > > > Comparing DejaGnu/GCC testsuite '*.sum' files between two systems ("old" > > vs. "new") that ought

Re: back to cvs, cool

2020-05-13 Thread Ian Lance Taylor via Gcc
On Wed, May 13, 2020 at 9:56 AM Mike Stump via Gcc wrote: > > As seen in recent bug report: > > CVS Commits 2020-05-12 20:40:40 UTC > > I guess that git thing was a bust and we're back to using cvs now. At least > Ian did up the remote patches to make cvs work better. In fairness, I worked

Re: dejagnu version update?

2020-05-13 Thread Rainer Orth
Jonathan Wakely via Gcc writes: >> I had previously approved the update to 1.5.3, but no one really wanted >> it as no one updated the requirement. Let's have the 1.6 discussion. >> I'm not only inclined to up to 1.6, but to actually edit it in this time. > > Would the tests actually refuse to

back to cvs, cool

2020-05-13 Thread Mike Stump via Gcc
As seen in recent bug report: CVS Commits 2020-05-12 20:40:40 UTC I guess that git thing was a bust and we're back to using cvs now. At least Ian did up the remote patches to make cvs work better.

Re: ChangeLog files - server and client scripts

2020-05-13 Thread Joseph Myers
On Wed, 13 May 2020, Martin Liška wrote: > I'm sending the gcc-changelog relates scripts which should be added to contrib > folder. The patch contains: > - git_check_commit.py - checking script that verifies git message format We need a documentation patch to contribute.html or gitwrite.html

Re: dejagnu version update?

2020-05-13 Thread Rob Savoye
On 5/13/20 10:51 AM, Mike Stump wrote: > So, now that ubuntu 20.04 is out and RHEL 8 is out, and they both > contain 6, and SLES has 6 and since we've been sitting at 1.4.4 for > so long, anyone want to not update dejagnu to require 1.6? We do still find and fix bugs occasionally. :-) And

Re: dejagnu version update?

2020-05-13 Thread Maciej W. Rozycki via Gcc
On Wed, 13 May 2020, Rainer Orth wrote: > > I'm in favour of requiring 1.5.3 or later, so 1.6 would be OK for me. > > If we go beyond 1.5.x, we need to go all the way up to 1.6.2: 1.6 and > 1.6.1 have an ugly bug that can miss timeouts, causing tests to hang > indefinitely until one manually

Re: how to find variable related to a virtual ssa name

2020-05-13 Thread 易会战 via Gcc
Thanks, I patch a version, it can give a heap variable info. Now the curious is the patch version cannot give right answer for variable length array. for example, int func(int m, int n) { int array[m][n]; ... } here array is allocated by alloca, obviously a local variable. But the original

Re: how to find variable related to a virtual ssa name

2020-05-13 Thread 易会战 via Gcc
There are some other cases that I cannot get right answer. case1: interproceduure func(int*arg) { return arg[0] + arg[1] } func2() { int a[10] return func(a); } here func cannot tell arg is local var. case 2: global array point to local int *array[3] int func(int x) { int sub1[10]; int sub2[10];

Re: how to find variable related to a virtual ssa name

2020-05-13 Thread Richard Biener via Gcc
On Wed, May 13, 2020 at 11:08 AM 易会战 wrote: > > yes, it does not escape the function, but indeed allocate memory on heap. > There is much specific method to judge the memory on heap although not escape > the function? Not at the moment. The info is computed by tree-ssa-structalias.c in

Re: Automatically generated ChangeLog files - PHASE 1

2020-05-13 Thread Richard Biener via Gcc
On Wed, May 13, 2020 at 11:27 AM Martin Liška wrote: > > On 5/13/20 10:16 AM, Richard Sandiford wrote: > > As far as this particular example goes, shouldn't the "testsuite/" line > > be dropped from the above? > > Good point. Fixes now with: > > $ ./git_email.py >

Re: how to find variable related to a virtual ssa name

2020-05-13 Thread 易会战 via Gcc
now I am working on gcc-9.3, can you give the specific code location to check not escaped heap? I try to add a flag. ---Original--- From: "Richard Biener"

DejaGnu/GCC testsuite behavior regarding multiple 'dg-do'

2020-05-13 Thread Thomas Schwinge
Hi! Comparing DejaGnu/GCC testsuite '*.sum' files between two systems ("old" vs. "new") that ought to return identical results, I found that they didn't: @@ -75032,6 +75023,7 @@ PASS: g++.dg/expr/bitfield4.C -std=c++98 (test for excess errors) PASS: g++.dg/expr/bitfield5.C -std=c++14

ChangeLog files - server and client scripts

2020-05-13 Thread Martin Liška
Hi. I'm sending the gcc-changelog relates scripts which should be added to contrib folder. The patch contains: - git_check_commit.py - checking script that verifies git message format - git_update_version.py - a replacement of maintainer-scripts/update_version_git which bumps DATESTAMP and

Re: size of exception handling (Was: performance of exception handling)

2020-05-13 Thread Jonathan Wakely via Gcc
On Tue, 12 May 2020, 10:15 Oleg Endo, wrote: > > On Tue, 2020-05-12 at 09:20 +0200, Freddie Chopin wrote: > > > > I actually have to build my own toolchain instead of the one provided > > by ARM, because to really NOT use C++ exceptions, you have to recompile > > the whole libstdc++ with

Re: how to find variable related to a virtual ssa name

2020-05-13 Thread 易会战 via Gcc
Which pass computes the points-to info, maybe There are some details? Do you know it? ---Original--- From: "Richard Biener"

Re: Automatically generated ChangeLog files - PHASE 1

2020-05-13 Thread Martin Liška
On 5/13/20 11:50 AM, Jozef Lawrynowicz wrote: "git_changelog.py" is nowhere to be found in the gcc-changelog or gcc-reposurgeon-8 repos. Is it equivalent to any of the other scripts in gcc-changelog? Yes, I made some refactoring and I'm going to send a proper gcc-patches submission. The

Re: how to find variable related to a virtual ssa name

2020-05-13 Thread Richard Biener via Gcc
On Wed, May 13, 2020 at 11:38 AM 易会战 wrote: > > now I am working on gcc-9.3, can you give the specific code location to check > not escaped heap? I try to add a flag. set_uids_in_ptset > ---Original--- > From: "Richard Biener" > Date: Wed, May 13, 2020 17:28 PM > To: "易会战"; > Cc: "gcc"; >

Re: Automatically generated ChangeLog files - PHASE 1

2020-05-13 Thread Martin Liška
On 5/13/20 10:16 AM, Richard Sandiford wrote: As far as this particular example goes, shouldn't the "testsuite/" line be dropped from the above? Good point. Fixes now with: $ ./git_email.py patches/0020-IPA-Avoid-segfault-in-devirtualization_time_bonus-PR.patch Errors: first line should

Re: Automatically generated ChangeLog files - PHASE 1

2020-05-13 Thread Martin Liška
On 5/13/20 11:29 AM, Richard Biener wrote: Hmm, it's OK in the commit but it should be omitted in the ChangeLog files. No, ChangeLog file identification should not be prepended with a tab. "testsuite/" will work. Martin

Re: Automatically generated ChangeLog files - PHASE 1

2020-05-13 Thread Richard Earnshaw
On 12/05/2020 10:05, Martin Liška wrote: > Hi. > > Thanks to Jakub, we finally set up an experimental environment: > gcc.gnu.org/home/gccadmin/gcc-reposurgeon-8.git > > The repository now contains a new pre-commit hook that validates > the git commit format ([1]) and provides a reasonable error

C-Vise: speed?

2020-05-13 Thread Martin Liška
Hello. I've made some measurements for GCC PRs that I've reduced recently: - PR92516 - C++ - 6.5MB C-Reduce: 77 minutes C-Vise: 35 minutes - PR94523 - C++ - 2.1MB C-Reduce: 33 minutes C-Vise: 15 minutes - PR94632 - C++ - 3.3MB C-Reduce: 28 minutes C-Vise: 20 minutes - PR94937 -

Re: ChangeLog files - server and client scripts

2020-05-13 Thread Richard Earnshaw
On 13/05/2020 12:05, Martin Liška wrote: > Hi. > > I'm sending the gcc-changelog relates scripts which should be added to > contrib > folder. The patch contains: > - git_check_commit.py - checking script that verifies git message format > - git_update_version.py - a replacement of >

Re: ChangeLog files - server and client scripts

2020-05-13 Thread Martin Liška
On 5/13/20 3:24 PM, Richard Earnshaw wrote: I've just realized this doesn't give us an easy way to mark changes for the root-level ChangeLog file, unless, perhaps "@@ CL ." works? This works fine: 'ChangeLog:' as seen for instance here: commit 9ad3c1d81c129fc76594b9df5b798c380cbf03ee Author:

Re: [PR 95013] EOF location is at end of file

2020-05-13 Thread Christophe Lyon via Gcc-patches
On Wed, 13 May 2020 at 02:24, H.J. Lu via Gcc-patches wrote: > > On Tue, May 12, 2020 at 2:24 PM Nathan Sidwell wrote: > > > > My recent C++ parser change to pay attention to EOF location uncovered a > > separate bug. The preprocesor's EOF logic would set the EOF location to > > be the

[Bug debug/95098] Out of scope variable visible during debugging at Og

2020-05-13 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95098 Richard Biener changed: What|Removed |Added CC||aoliva at gcc dot gnu.org,

[Bug preprocessor/95013] [11 Regression] FAIL: obj-c++.dg/property/property-neg-6.mm syntax-error-10.mm

2020-05-13 Thread cvs-commit at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95013 --- Comment #9 from CVS Commits --- The master branch has been updated by Christophe Lyon : https://gcc.gnu.org/g:a7b7818f3dbd14cc7577d25dcebaded07395c476 commit r11-346-ga7b7818f3dbd14cc7577d25dcebaded07395c476 Author: Christophe Lyon Date:

[Bug tree-optimization/92177] [10 Regression] gcc.dg/vect/bb-slp-22.c FAILs

2020-05-13 Thread cvs-commit at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=92177 --- Comment #13 from CVS Commits --- The releases/gcc-10 branch has been updated by Richard Biener : https://gcc.gnu.org/g:ff9d4e09566e24d4dff10adeaef109823266c7bd commit r10-8140-gff9d4e09566e24d4dff10adeaef109823266c7bd Author: Richard

[Bug c++/95100] New: xxx_view adaptors don't work with pipeline operator

2020-05-13 Thread rhalbersma at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95100 Bug ID: 95100 Summary: xxx_view adaptors don't work with pipeline operator Product: gcc Version: 10.0 Status: UNCONFIRMED Severity: normal Priority:

[Bug tree-optimization/95019] Optimizer produces suboptimal code related to -ftree-ivopts

2020-05-13 Thread zhongyunde at tom dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95019 --- Comment #2 from zhongyunde at tom dot com --- It is a generic issue for all targets, such as x86, it also don't enpand IVOPTs as index is not used for DEST and Src directly. we may need expand IVOPTs, then different targets can select

[Bug ipa/94947] [8/9/10 Regression] -fipa-pta + pthread_once crash since r6-5684-g47e5754e17e9ac3b

2020-05-13 Thread cvs-commit at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94947 --- Comment #9 from CVS Commits --- The releases/gcc-10 branch has been updated by Richard Biener : https://gcc.gnu.org/g:a68d4b47a6b5b23a5d7ab3b358f559f41568512f commit r10-8142-ga68d4b47a6b5b23a5d7ab3b358f559f41568512f Author: Richard Biener

Re: [PR77691] x86-vxworks malloc aligns to 8 bytes like solaris

2020-05-13 Thread Alexandre Oliva
Hello, Jonathan, On May 9, 2020, Jonathan Wakely wrote: > On 08/05/20 17:22 -0300, Alexandre Oliva wrote: >> (Couldn't r1->allocate(2, alignof(char)) possibly return a pointer >> that's *not* aligned? Maybe we should drop the test even >> if !defined(BAD_MAX_ALIGN_T).) > Yes. > Different

[PATCH] Fix -fcompare-debug issue in purge_dead_edges [PR95080]

2020-05-13 Thread Jakub Jelinek via Gcc-patches
Hi! The following testcase fails with -fcompare-debug, the bug used to be latent since introduction of -fcompare-debug. The loop at the start of purge_dead_edges behaves differently between -g0 and -g - if the last insn is a DEBUG_INSN, then it skips not just DEBUG_INSNs but also NOTEs until it

Re: [PATCH] Fix -fcompare-debug issue in purge_dead_edges [PR95080]

2020-05-13 Thread Richard Biener
On Wed, 13 May 2020, Jakub Jelinek wrote: > Hi! > > The following testcase fails with -fcompare-debug, the bug used to be latent > since introduction of -fcompare-debug. > The loop at the start of purge_dead_edges behaves differently between -g0 > and -g - if the last insn is a DEBUG_INSN, then

[Bug tree-optimization/95051] error: invalid RHS for gimple memory store:

2020-05-13 Thread cvs-commit at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95051 --- Comment #8 from CVS Commits --- The master branch has been updated by Martin Liska : https://gcc.gnu.org/g:3d96f7b92415b7a277a87e7825efc958030e20b6 commit r11-348-g3d96f7b92415b7a277a87e7825efc958030e20b6 Author: Martin Liska Date: Wed

[Bug target/95018] [10/11 Regression] Excessive unrolling for Fortran library array handling

2020-05-13 Thread tkoenig at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95018 --- Comment #29 from Thomas Koenig --- It is also interesting that this variant --- a/libgfortran/generated/in_pack_i4.c +++ b/libgfortran/generated/in_pack_i4.c @@ -88,7 +88,7 @@ internal_pack_4 (gfc_array_i4 * source) count[0]++;

[Bug testsuite/95092] new test case gcc.dg/asan/pr95051.c fails

2020-05-13 Thread marxin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95092 Martin Liška changed: What|Removed |Added Resolution|--- |FIXED Status|ASSIGNED

Re: [PATCH] Add -fsplit-dwarf

2020-05-13 Thread Richard Biener via Gcc-patches
On Wed, May 13, 2020 at 1:40 AM Fangrui Song via Gcc-patches wrote: > > -fsplit-dwarf is similar to -gsplit-dwarf, but does not enable debugging > information by itself. This makes it easier to be plugged into a build > system without worrying that unnecessary debugging information may be >

[Bug ipa/94947] [8/9/10 Regression] -fipa-pta + pthread_once crash since r6-5684-g47e5754e17e9ac3b

2020-05-13 Thread cvs-commit at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94947 --- Comment #8 from CVS Commits --- The releases/gcc-10 branch has been updated by Richard Biener : https://gcc.gnu.org/g:67d00c438285ecf9b8e96c12d1f6b05fd2ea3c21 commit r10-8141-g67d00c438285ecf9b8e96c12d1f6b05fd2ea3c21 Author: Richard Biener

[Bug fortran/94690] [OpenMP] omp ... distribute – lastprivate not permitted and more issues

2020-05-13 Thread cvs-commit at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94690 --- Comment #9 from CVS Commits --- The master branch has been updated by Tobias Burnus : https://gcc.gnu.org/g:f884bef2105d748fd7869cd641cbb4f6b6bb commit r11-349-gf884bef2105d748fd7869cd641cbb4f6b6bb Author: Tobias Burnus Date:

[PATCH] tree-optimization/33315 - common stores during sinking

2020-05-13 Thread Richard Biener
This implements commoning of stores to a common successor in a simple ad-hoc way. I've decided to put it into the code sinking pass since, well, it sinks stores. It's still separate since it does not really sink code into less executed places. It's ad-hoc since it does not perform any

Re: [PATCH] Fold single imm use of a FMA if it is a negation [PR95060]

2020-05-13 Thread Richard Biener
On Wed, 13 May 2020, Jakub Jelinek wrote: > Hi! > > match.pd already has simplifications for negation of a FMA (FMS, FNMA, FNMS) > call if it is single use, but when the widening_mul pass discovers FMAs, > nothing folds the statements anymore. > > So, the following patch adjusts the

[Bug debug/95098] New: Out of scope variable visible during debugging at Og

2020-05-13 Thread massarelli at diag dot uniroma1.it
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95098 Bug ID: 95098 Summary: Out of scope variable visible during debugging at Og Product: gcc Version: 11.0 Status: UNCONFIRMED Severity: normal Priority: P3

[Bug tree-optimization/95097] Missed optimization with bitfield value ranges

2020-05-13 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95097 --- Comment #3 from Richard Biener --- Just to quote EVRP sees : _1 = VIEW_CONVERT_EXPR(f); _2 = _1 & 1048575; if (_2 != 0) goto ; [INV] else goto ; [INV] : _3 = f.x; _4 = (unsigned int) _3; y_8 = _4 * 4096; if

[Bug tree-optimization/95019] Optimizer produces suboptimal code related to -ftree-ivopts

2020-05-13 Thread amker at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95019 --- Comment #3 from bin cheng --- (In reply to zhongyu...@tom.com from comment #2) > It is a generic issue for all targets, such as x86, it also don't enpand Yes, as said it's because SCEV currently doesn't model this, so it's not target

Re: testsuite: Fix up gcc.dg/asan/pr95051.c testcase [PR95051]

2020-05-13 Thread Martin Liška
On 5/12/20 9:23 PM, Jakub Jelinek wrote: Hi! On Tue, May 12, 2020 at 12:06:25PM -0700, H.J. Lu wrote: Excess errors: cc1: error: '-fsanitize=address' is incompatible with '-fsanitize=kernel-address' asan.exp adds -fsanitize=address which is incompatible with -fsanitize=kernel-address, so we

[Bug libgcc/95099] New: Build a cross compiler for m32c on Windows (Cygwin)

2020-05-13 Thread dj_adilovic at hotmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95099 Bug ID: 95099 Summary: Build a cross compiler for m32c on Windows (Cygwin) Product: gcc Version: 10.1.0 Status: UNCONFIRMED Severity: normal Priority: P3

[Bug libstdc++/77691] [8/9/10/11 regression] experimental/memory_resource/resource_adaptor.cc FAILs

2020-05-13 Thread cvs-commit at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77691 --- Comment #42 from CVS Commits --- The master branch has been updated by Alexandre Oliva : https://gcc.gnu.org/g:883246530f1bb10d854f455e1c3d55b93675690a commit r11-347-g883246530f1bb10d854f455e1c3d55b93675690a Author: Alexandre Oliva Date:

[Bug target/95018] [10/11 Regression] Excessive unrolling for Fortran library array handling

2020-05-13 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95018 --- Comment #30 from Richard Biener --- (In reply to Thomas Koenig from comment #29) > It is also interesting that this variant > > --- a/libgfortran/generated/in_pack_i4.c > +++ b/libgfortran/generated/in_pack_i4.c > @@ -88,7 +88,7 @@

[Bug target/95018] [10/11 Regression] Excessive unrolling for Fortran library array handling

2020-05-13 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95018 --- Comment #28 from Richard Biener --- > It the growth limit seems could be refined. The ^ is an exponent operation, > right? Yes. The idea is to limit growth more when there is no benefit of unrolling detected by the cost model (which

[Bug tree-optimization/95097] Missed optimization with bitfield value ranges

2020-05-13 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95097 Andrew Pinski changed: What|Removed |Added Blocks|85316 | Severity|normal

[Bug tree-optimization/92177] [10 Regression] gcc.dg/vect/bb-slp-22.c FAILs

2020-05-13 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=92177 Richard Biener changed: What|Removed |Added Resolution|--- |FIXED Status|ASSIGNED

[PATCH] Fold single imm use of a FMA if it is a negation [PR95060]

2020-05-13 Thread Jakub Jelinek via Gcc-patches
Hi! match.pd already has simplifications for negation of a FMA (FMS, FNMA, FNMS) call if it is single use, but when the widening_mul pass discovers FMAs, nothing folds the statements anymore. So, the following patch adjusts the widening_mul pass to handle that. I had to adjust quite a lot of

[Bug rtl-optimization/95102] New: missed if-conversion

2020-05-13 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95102 Bug ID: 95102 Summary: missed if-conversion Product: gcc Version: 11.0 Status: UNCONFIRMED Severity: normal Priority: P3 Component: rtl-optimization

[Bug rtl-optimization/95102] missed if-conversion

2020-05-13 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95102 --- Comment #1 from Richard Biener --- OK, so one reason is that if (!can_conditionally_move_p (x_mode)) return FALSE; returns false for E_V4SFmode on x86. min/max detection is based on fp_cmov expansion for scalar FP on x86 though

[RFC PATCH v2] cgraph support for late declare variant resolution

2020-05-13 Thread Jakub Jelinek via Gcc-patches
Hi! This is a new version of the https://gcc.gnu.org/legacy-ml/gcc-patches/2019-11/msg01493.html patch. Unlike the previous version, this one actually works properly except for LTO, bootstrapped/regtested on x86_64-linux and i686-linux too. In short, #pragma omp declare variant is a directive

RE: [GCC][PATCH][ARM]: Fix the wrong code-gen generated by MVE vector load/store intrinsics (PR94959).

2020-05-13 Thread Srinath Parvathaneni
Hi, > -Original Message- > From: Christophe Lyon > Sent: 13 May 2020 11:20 > To: Srinath Parvathaneni > Cc: gcc Patches ; Richard Earnshaw > > Subject: Re: [GCC][PATCH][ARM]: Fix the wrong code-gen generated by MVE > vector load/store intrinsics (PR94959). > > Hi, > > > On Wed, 13

[PATCH] Remove SLP_INSTANCE_GROUP_SIZE

2020-05-13 Thread Richard Biener
This removes the SLP_INSTANCE_GROUP_SIZE member since the number of lanes throughout a SLP subgraph is not necessarily constant. Bootstrapped on x86_64-unknown-linux-gnu, testing in progress. Richard. 2020-05-13 Richard Biener * tree-vectorizer.h (SLP_INSTANCE_GROUP_SIZE): Remove.

Re: [GCC][PATCH][ARM]: Fix the wrong code-gen generated by MVE vector load/store intrinsics (PR94959).

2020-05-13 Thread Christophe Lyon via Gcc-patches
On Wed, 13 May 2020 at 13:45, Srinath Parvathaneni wrote: > > Hi, > > > -Original Message- > > From: Christophe Lyon > > Sent: 13 May 2020 11:20 > > To: Srinath Parvathaneni > > Cc: gcc Patches ; Richard Earnshaw > > > > Subject: Re: [GCC][PATCH][ARM]: Fix the wrong code-gen generated

Re: [PR 95013] EOF location is at end of file

2020-05-13 Thread Nathan Sidwell
On 5/13/20 2:44 AM, Christophe Lyon wrote: On Wed, 13 May 2020 at 02:24, H.J. Lu via Gcc-patches [PR 95013] Fix gcc.dg/unclosed-init.c 2020-05-13 Christophe Lyon PR preprocessor/95013 * gcc.dg/unclosed-init.c: Add missing comment in dg-error. Thanks for

[Bug target/95099] Build a cross compiler for m32c on Windows (Cygwin)

2020-05-13 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95099 Andrew Pinski changed: What|Removed |Added Resolution|--- |DUPLICATE

[Bug target/83670] [8/9/10/11 Regression] m32c ICE in leaf_function_p, at final.c:4368

2020-05-13 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83670 Andrew Pinski changed: What|Removed |Added Target Milestone|--- |8.5 Summary|m32c ICE in

[Bug target/83670] m32c ICE in leaf_function_p, at final.c:4368

2020-05-13 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83670 Andrew Pinski changed: What|Removed |Added CC||dj_adilovic at hotmail dot com ---

Re: ChangeLog files - server and client scripts

2020-05-13 Thread Martin Liška
The scripts were just installed to master except the git alias. I'm sending that in a separate patch. Now the alias can be used from any subfolder in a gcc git repository. Martin >From eb47191e8d8cbbda285c4df7eb2d1e98091edab9 Mon Sep 17 00:00:00 2001 From: Martin Liska Date: Wed, 13 May 2020

[Bug middle-end/95052] [9/10/11 Regression] Excess padding of partially initialized strings/char arrays

2020-05-13 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95052 Andrew Pinski changed: What|Removed |Added Target Milestone|--- |9.4 Summary|Excess padding

[Bug tree-optimization/95034] Failure to convert xor pattern (made out of or+and) to xor

2020-05-13 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95034 Andrew Pinski changed: What|Removed |Added Severity|normal |enhancement

[PATCH] Add default revisions argument for git_check_commit.py.

2020-05-13 Thread Martin Liška
A small tweak to the script that I'm going to install. Martin contrib/ChangeLog: 2020-05-13 Martin Liska * gcc-changelog/git_check_commit.py: Add default argument HEAD for revisions and improve error message output. --- contrib/gcc-changelog/git_check_commit.py | 6 +++---

[Bug fortran/95104] Segfault on a legal WAIT statement

2020-05-13 Thread longb at cray dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95104 --- Comment #1 from Bill Long --- The program appears to be legal and should execute and print 0. The last paragraph of 12.7.2 WAIT statement (current Fortran standard) is Execution of a WAIT statement specifying a unit that does not exist,

[Bug fortran/91731] Configure error on building MPICH

2020-05-13 Thread keroro.90 at hotmail dot it
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91731 keroro.90 at hotmail dot it changed: What|Removed |Added CC||keroro.90 at hotmail dot it

Re: [PATCH PR94969]Add unit distant vector to DDR in case of invariant access functions

2020-05-13 Thread Christophe Lyon via Gcc-patches
Hi Bin, On Mon, 11 May 2020 at 14:54, Richard Biener via Gcc-patches wrote: > > On Mon, May 11, 2020 at 7:52 AM bin.cheng via Gcc-patches > wrote: > > > > Hi, > > As analyzed in PR94969, data dependence analysis now misses dependence > > vector for specific case in which DRs in DDR have the

Re: [PATCH] x86: Properly count cost of XMM register push

2020-05-13 Thread Uros Bizjak via Gcc-patches
On Wed, May 13, 2020 at 1:05 PM Uros Bizjak wrote: > > On Tue, May 12, 2020 at 10:07 PM H.J. Lu wrote: > > > > Update STV pass to properly count cost of XMM register push. In 32-bit > > mode, to convert XMM register push in DImode, we do an XMM store in > > DImode, followed by 2 memory pushes

ChangeLog files - server and client scripts (git cherry-pick)

2020-05-13 Thread Martin Liška
On 5/13/20 1:05 PM, Martin Liška wrote: I suggest to use native 'git revert XYZ' and 'git cherry-pick -x XYZ'. I've prepared a working version of Revert format: https://github.com/marxin/gcc-changelog/tree/cherry-pick So using git cherry-pick -x HASH one gets something like: $ cat

[Bug c++/95103] New: Unexpected -Wclobbered in bits/vector.tcc with -O2

2020-05-13 Thread sbergman at redhat dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95103 Bug ID: 95103 Summary: Unexpected -Wclobbered in bits/vector.tcc with -O2 Product: gcc Version: unknown Status: UNCONFIRMED Severity: normal Priority: P3

Re: [PATCH] x86: Properly count cost of XMM register push

2020-05-13 Thread H.J. Lu via Gcc-patches
On Wed, May 13, 2020 at 5:04 AM Uros Bizjak wrote: > > On Wed, May 13, 2020 at 1:05 PM Uros Bizjak wrote: > > > > On Tue, May 12, 2020 at 10:07 PM H.J. Lu wrote: > > > > > > Update STV pass to properly count cost of XMM register push. In 32-bit > > > mode, to convert XMM register push in

[Bug fortran/95104] New: Segfault on a legal WAIT statement

2020-05-13 Thread longb at cray dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95104 Bug ID: 95104 Summary: Segfault on a legal WAIT statement Product: gcc Version: 9.3.0 Status: UNCONFIRMED Severity: normal Priority: P3 Component: fortran

[PATCH] add vectype parameter to add_stmt_cost hook

2020-05-13 Thread Richard Biener
This adds a vectype parameter to add_stmt_cost which avoids the need to pass down a (wrong) stmt_info just to carry this information. Useful for invariants which do not have a stmt_info associated. Bootstrap and regtest running on x86_64-unknown-linux-gnu. Richard. 2020-05-13 Richard Biener

[PATCH] [V2] rs6000: Add vec_extracth and vec_extractl

2020-05-13 Thread Bill Schmidt via Gcc-patches
From: Kelvin Nilsen Add new insns vextdu[bhw]vlx, vextddvlx, vextdu[bhw]vhx, and vextddvhx, along with built-in access and overloaded built-in access to these insns. Changes from previous patch: * Removed the int iterators * Created separate expansions and insns vextractl

Ping: [PATCH] wwwdocs: Add D front-end section for GCC 10 changes

2020-05-13 Thread Iain Buclaw via Gcc-patches
Ping. On 07/05/2020 16:04, Iain Buclaw via Gcc-patches wrote: > Hi, > > Updated the patch to include missed changes, and slighted reworded some > entries > to make them clearer/read easier. > > OK to commit? > > Iain. > > --- > htdocs/gcc-10/changes.html | 35

Re: [PATCH] contrib/vimrc: Reduce textwidth for commit messages

2020-05-13 Thread Martin Liška
On 5/4/20 8:18 PM, Martin Liška wrote: I support the patch, And as there's no feedback I also installed the patch. Martin

libbacktrace patch committed: Mark state unused in ztest.c test_large

2020-05-13 Thread Ian Lance Taylor via Gcc-patches
This libbacktrace patch marks the state parameter of test_large in ztest.c as ATTRIBUTE_UNUSED. The parameter is not used if HAVE_ZLIB is not defined. Bootstrapped and ran libbacktrace tests on x86_64-pc-linux-gnu. Committed to mainline. Ian 2020-05-13 Ian Lance Taylor * ztest.c

[Bug c++/95111] coroutines use-after-free with lambdas

2020-05-13 Thread iains at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95111 --- Comment #4 from Iain Sandoe --- (In reply to Avi Kivity from comment #3) > The test case I uploaded only shows the failure, it won't work if gcc worked > as I expect it. I'll try to get a better testcase, unfortunately a small > coroutine

[C++] some cleanup patches

2020-05-13 Thread Nathan Sidwell
I've committed this set of minor cleanups from the modules branch. nathan -- Nathan Sidwell 2020-05-13 Nathan Sidwell Formatting fixups & some simplifications. * pt.c (spec_hash_table): New typedef. (decl_specializations, type_specializations): Use it. (retrieve_specialization):

[C++] simplify typedef access checking

2020-05-13 Thread Nathan Sidwell
I discovered template typedef access checking was more expensive than it need be. The call of get_types_needed_access_check in the FOR_EACH_VEC_SAFE_ELT is the moral equivalent of for (size_t pos = 0; pos != strlen (string); pos++)' Let's not do that. nathan -- Nathan Sidwell 2020-05-13

[Bug lto/64636] LTO PGO bootstrap fails on linux-sparc64 in stream_out_histogram_value

2020-05-13 Thread ostash at ostash dot kiev.ua
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64636 Viktor Ostashevskyi changed: What|Removed |Added CC||ostash at ostash dot kiev.ua ---

[Bug target/95112] New: i386 procedures have prolog endbr32

2020-05-13 Thread akobets at mail dot ru
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95112 Bug ID: 95112 Summary: i386 procedures have prolog endbr32 Product: gcc Version: 9.3.0 Status: UNCONFIRMED Severity: normal Priority: P3 Component: target

[Bug middle-end/61577] [4.9.0 Regression] can't compile on hp-ux v3 ia64

2020-05-13 Thread jared.martinsen at fiserv dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61577 Jared changed: What|Removed |Added CC||jared.martinsen at fiserv dot com --- Comment

[Bug tree-optimization/95113] [10/11 Regression] Wrong code w/ -O2 -fexceptions -fnon-call-exceptions

2020-05-13 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95113 Jakub Jelinek changed: What|Removed |Added Last reconfirmed||2020-05-13 CC|

[Bug middle-end/95108] [9/10/11 Regression] ICE in tree_fits_uhwi_p, at tree.c:7292

2020-05-13 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95108 Jakub Jelinek changed: What|Removed |Added Assignee|unassigned at gcc dot gnu.org |jakub at gcc dot gnu.org

Re: [PATCH] c++: SFINAE for invalid delete-expression [PR79706]

2020-05-13 Thread Jason Merrill via Gcc-patches
On 5/12/20 11:36 PM, Patrick Palka wrote: This fixes SFINAE when substitution yields an invalid delete-expression due to the pertinent deallocation function being marked deleted or otherwise inaccessible. We need to check for an erroneous result from build_op_delete_call and exit early in that

[Bug c++/95111] coroutines use-after-free with lambdas

2020-05-13 Thread iains at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95111 --- Comment #6 from Iain Sandoe --- (In reply to Avi Kivity from comment #5) > This snippet from cppreference: > > If the coroutine is a non-static member function, such as task > my_class::method1(int x) const;, its Promise type is >

  1   2   3   >