https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98533
Roman Zhuykov changed:
What|Removed |Added
CC||zhroma at gcc dot gnu.org
--- Comment
: c++
Assignee: unassigned at gcc dot gnu.org
Reporter: zhroma at gcc dot gnu.org
Target Milestone: ---
$ cat test.cc
extern long global;
int foo (unsigned x) {
return x == ((global ? 64 : 32) - 1) / 8;
}
$ gcc -x c -c -Werror=sign-compare test.cc
$ gcc -x c -c -Werror=sign
,
||jakub at redhat dot com,
||zhroma at gcc dot gnu.org
--- Comment #1 from Roman Zhuykov ---
> Is it an issue at all?
> Should I report more issues of this kind in the future?
I think the answer is "ye
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=100647
--- Comment #1 from Roman Zhuykov ---
Thanks, I'll take a closer look in a few days, but this should be a dup of
PR93264 (see also PR90040), which doesn't have any easy solution...
at gcc dot gnu.org |zhroma at gcc dot
gnu.org
Resolution|--- |FIXED
--- Comment #10 from Roman Zhuykov ---
Now fixed on all branches.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=100225
--- Comment #4 from Roman Zhuykov ---
Created attachment 50704
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=50704=edit
Tested patch
Fix in https://gcc.gnu.org/pipermail/gcc-patches/2021-April/569110.html,
pushing to trunk tomorrow
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97421
Roman Zhuykov changed:
What|Removed |Added
Resolution|--- |FIXED
Status|NEW
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97421
--- Comment #8 from Roman Zhuykov ---
*** Bug 97440 has been marked as a duplicate of this bug. ***
||zhroma at gcc dot gnu.org
Status|UNCONFIRMED |RESOLVED
--- Comment #2 from Roman Zhuykov ---
Same issue as PR97421
*** This bug has been marked as a duplicate of bug 97421 ***
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97851
Roman Zhuykov changed:
What|Removed |Added
Resolution|--- |DUPLICATE
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97421
--- Comment #7 from Roman Zhuykov ---
*** Bug 97851 has been marked as a duplicate of this bug. ***
|NEW
Assignee|unassigned at gcc dot gnu.org |zhroma at gcc dot
gnu.org
--- Comment #6 from Roman Zhuykov ---
Thanks for catching this!
When modulo sched succeded and found that counter register is const-initialized
in prologue it wrongly directly adjust the initization without
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97421
Roman Zhuykov changed:
What|Removed |Added
CC||zhroma at gcc dot gnu.org
--- Comment
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94775
--- Comment #15 from Roman Zhuykov ---
(In reply to Marek Polacek from comment #14)
> Patch reverted from 10/11.
Maybe I've missed something in gcc-patches, but why it was reverted? I can
imagine reasons to revert in 10 branch, but in master...
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94775
--- Comment #1 from Roman Zhuykov ---
Additionally checked that vanilla gcc-9.3.0 version is also affected.
Priority: P3
Component: tree-optimization
Assignee: unassigned at gcc dot gnu.org
Reporter: zhroma at gcc dot gnu.org
Target Milestone: ---
At least ubuntu v8.3.0 and trunk v10.0.1 (20200402) ICE on Linux x86-64 with
the following reduced testcase while emitting
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93264
--- Comment #11 from Roman Zhuykov ---
(In reply to Richard Biener from comment #9)
Thank you, I'm glad to see new ideas and some discussion.
> On the testcase itself
>
> diff --git a/gcc/modulo-sched.c b/gcc/modulo-sched.c
> index
Priority: P3
Component: debug
Assignee: unassigned at gcc dot gnu.org
Reporter: zhroma at gcc dot gnu.org
Target Milestone: ---
This command:
gcc -c -fcompare-debug -O gcc/testsuite/g++.dg/cpp1z/nodiscard3.C
fails on master (r10-7374) and 9 branch
Priority: P3
Component: c++
Assignee: unassigned at gcc dot gnu.org
Reporter: zhroma at gcc dot gnu.org
Target Milestone: ---
This commands:
gcc -c -O0 -std=c++20 -fcompare-debug
gcc/testsuite/g++.dg/cpp1z/init-statement6.C
gcc -c -O0 -std=c++2a -fcompare
Priority: P3
Component: tree-optimization
Assignee: unassigned at gcc dot gnu.org
Reporter: zhroma at gcc dot gnu.org
Target Milestone: ---
This command:
gcc -c -O2 -fcompare-debug gcc/testsuite/gcc.dg/tree-ssa/phi-opt-13.c
fails on todays master (r10-7237) on ppc64le
Component: sanitizer
Assignee: unassigned at gcc dot gnu.org
Reporter: zhroma at gcc dot gnu.org
CC: dodji at gcc dot gnu.org, dvyukov at gcc dot gnu.org,
jakub at gcc dot gnu.org, kcc at gcc dot gnu.org, marxin at
gcc dot gnu.org
Target Milestone
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94189
Roman Zhuykov changed:
What|Removed |Added
Summary|[9/10 Regression] |[9/10 Regression]
Priority: P3
Component: rtl-optimization
Assignee: unassigned at gcc dot gnu.org
Reporter: zhroma at gcc dot gnu.org
CC: jakub at gcc dot gnu.org
Target Milestone: ---
This command:
gcc -c -O2 -fcompare-debug gcc/testsuite/gcc.dg/Wstrict-overflow-22
Priority: P3
Component: tree-optimization
Assignee: unassigned at gcc dot gnu.org
Reporter: zhroma at gcc dot gnu.org
Target Milestone: ---
Running
gcc -O2 -fcompare-debug gcc/testsuite/gcc.c-torture/compile/pr71109.c
fails with recent master on powerpc64le.
I also
Priority: P3
Component: tree-optimization
Assignee: unassigned at gcc dot gnu.org
Reporter: zhroma at gcc dot gnu.org
CC: jakub at redhat dot com, linkw at gcc dot gnu.org
Target Milestone: ---
Target: powerpc64le-*-linux-gnu
I've
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=92410
Roman Zhuykov changed:
What|Removed |Added
CC||zhroma at gcc dot gnu.org
--- Comment
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93264
Roman Zhuykov changed:
What|Removed |Added
CC||amonakov at gcc dot gnu.org
--- Comment
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90040
--- Comment #3 from Roman Zhuykov ---
(In reply to Roman Zhuykov from comment #2)
> Same ICE also appears when compiling gcc.c-torture/execute/pr71550.c with
So, I've opened separate PR93264 for that example, and now we have some related
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93264
--- Comment #5 from Roman Zhuykov ---
Created attachment 47820
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=47820=edit
Considered "moving sms earlier" patch
I haven't tested "moving sms earlier" patch since 2011. But I remember there
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93264
Roman Zhuykov changed:
What|Removed |Added
CC||abel at gcc dot gnu.org
--- Comment #2
: UNCONFIRMED
Severity: normal
Priority: P3
Component: rtl-optimization
Assignee: unassigned at gcc dot gnu.org
Reporter: zhroma at gcc dot gnu.org
Blocks: 90040
Target Milestone: ---
Opening bug 90040 comment 2 as a separate regression PR
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93088
Roman Zhuykov changed:
What|Removed |Added
CC||zhroma at gcc dot gnu.org
--- Comment
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93053
Roman Zhuykov changed:
What|Removed |Added
Status|UNCONFIRMED |NEW
Last reconfirmed|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93053
--- Comment #4 from Roman Zhuykov ---
Maybe this should be catched earlier in configure scripts?
Are there any simple workaround without patching the gcc source? I'm not
familiar with different armvN -march option values. Probably it can help
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93053
--- Comment #1 from Roman Zhuykov ---
Now I can confirm it started with r275967.
: libgcc
Assignee: unassigned at gcc dot gnu.org
Reporter: zhroma at gcc dot gnu.org
Target Milestone: ---
On cfarm gcc115 machine (aarch64): using trunk source (r279717), running
"./configure --prefix=... --enable-languages=c,c++,fortran; make -j3" got:
/home/zh
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90040
--- Comment #2 from Roman Zhuykov ---
Same ICE also appears when compiling gcc.c-torture/execute/pr71550.c with -O3
-fomit-frame-pointer -funroll-loops -fpeel-loops -ftracer -finline-functions
-fmodulo-sched -freorder-blocks-and-partition on
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=92591
Roman Zhuykov changed:
What|Removed |Added
Status|ASSIGNED|RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=2019
--- Comment #12 from Roman Zhuykov ---
Author: zhroma
Date: Fri Dec 20 15:52:02 2019
New Revision: 279647
URL: https://gcc.gnu.org/viewcvs?rev=279647=gcc=rev
Log:
Backport: rename tests to reference the correct PR
2019-12-20 Roman Zhuykov
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=2019
--- Comment #11 from Roman Zhuykov ---
Author: zhroma
Date: Fri Dec 20 15:50:44 2019
New Revision: 279646
URL: https://gcc.gnu.org/viewcvs?rev=279646=gcc=rev
Log:
Backport: rename tests to reference the correct PR
2019-12-20 Roman Zhuykov
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=2019
--- Comment #10 from Roman Zhuykov ---
Author: zhroma
Date: Fri Dec 20 15:40:46 2019
New Revision: 279645
URL: https://gcc.gnu.org/viewcvs?rev=279645=gcc=rev
Log:
Rename tests to reference the correct PR
2019-12-20 Roman Zhuykov
*
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=92591
--- Comment #11 from Roman Zhuykov ---
Author: zhroma
Date: Fri Dec 20 15:07:58 2019
New Revision: 279643
URL: https://gcc.gnu.org/viewcvs?rev=279643=gcc=rev
Log:
modulo-sched: backport PR92591 fix from mainline
2019-12-20 Roman Zhuykov
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=92591
--- Comment #10 from Roman Zhuykov ---
Author: zhroma
Date: Fri Dec 20 15:05:12 2019
New Revision: 279642
URL: https://gcc.gnu.org/viewcvs?rev=279642=gcc=rev
Log:
modulo-sched: backport PR92591 fix from mainline
2019-12-20 Roman Zhuykov
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90040
Roman Zhuykov changed:
What|Removed |Added
CC||hubicka at gcc dot gnu.org,
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=92591
--- Comment #9 from Roman Zhuykov ---
Started discussion in mailing list:
https://gcc.gnu.org/ml/gcc-patches/2019-12/msg01223.html
|--- |FIXED
Assignee|unassigned at gcc dot gnu.org |zhroma at gcc dot
gnu.org
--- Comment #8 from Roman Zhuykov ---
Fixed.
Yes, I'm closing the PR, I suppose this should not be backported. It's a pity
we don't have an ability to create regression test
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=92591
--- Comment #7 from Roman Zhuykov ---
Author: zhroma
Date: Fri Dec 13 17:33:38 2019
New Revision: 279377
URL: https://gcc.gnu.org/viewcvs?rev=279377=gcc=rev
Log:
modulo-sched: fix branch rescheduling issue (PR92591)
PR
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90001
--- Comment #6 from Roman Zhuykov ---
Author: zhroma
Date: Fri Dec 13 17:02:53 2019
New Revision: 279375
URL: https://gcc.gnu.org/viewcvs?rev=279375=gcc=rev
Log:
modulo-sched: speed up DDG analysis (PR90001)
PR rtl-optimization/90001
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=92591
--- Comment #6 from Roman Zhuykov ---
Patches are still testing. Arm (both 32 and 64 bit) cross compilers are OK,
and I want to test some additional scenarios for powerpc. Probably I'll post
patches next week.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=92591
Roman Zhuykov changed:
What|Removed |Added
Attachment #47379|0 |1
is obsolete|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90264
--- Comment #6 from Roman Zhuykov ---
Ok, this seems quite clear from compiler developer point of view.
But I still want to add, that for compiler user, who knows how asprintf
function works, "Line A" version is correct and warning seems
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=92591
--- Comment #3 from Roman Zhuykov ---
Created attachment 47380
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=47380=edit
Parameters patch
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=92591
--- Comment #2 from Roman Zhuykov ---
Created attachment 47379
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=47379=edit
Proposed patch
||2019-11-27
Assignee|unassigned at gcc dot gnu.org |zhroma at gcc dot
gnu.org
Ever confirmed|0 |1
--- Comment #1 from Roman Zhuykov ---
Oops, it seems that non-zero sms-dfa-history is untested since modulo-sched was
added in 2004
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90264
Roman Zhuykov changed:
What|Removed |Added
Status|RESOLVED|REOPENED
Last reconfirmed|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86172
Bug 86172 depends on bug 90264, which changed state.
Bug 90264 Summary: [9/10 Regression] -Wnull-dereference QoI issue
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90264
What|Removed |Added
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90264
--- Comment #3 from Roman Zhuykov ---
(In reply to Jeffrey A. Law from comment #1)
> We set *seq = 0.
>
> Assume we do not return -1 from line A.
>
> The for loop's initial test will be false because out = *seq = 0
>
> out - *seq must also
Severity: normal
Priority: P3
Component: tree-optimization
Assignee: unassigned at gcc dot gnu.org
Reporter: zhroma at gcc dot gnu.org
CC: jakub at gcc dot gnu.org, law at redhat dot com, rguenth at
gcc dot gnu.org
Blocks
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87979
--- Comment #5 from Roman Zhuykov ---
Author: zhroma
Date: Fri Apr 26 16:04:54 2019
New Revision: 270609
URL: https://gcc.gnu.org/viewcvs?rev=270609=gcc=rev
Log:
Backport modulo-sched fixes from mainline
2019-04-23 Roman Zhuykov
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84032
--- Comment #7 from Roman Zhuykov ---
Author: zhroma
Date: Fri Apr 26 16:04:54 2019
New Revision: 270609
URL: https://gcc.gnu.org/viewcvs?rev=270609=gcc=rev
Log:
Backport modulo-sched fixes from mainline
2019-04-23 Roman Zhuykov
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87979
Roman Zhuykov changed:
What|Removed |Added
Status|UNCONFIRMED |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87979
--- Comment #3 from Roman Zhuykov ---
Author: zhroma
Date: Tue Apr 23 13:14:57 2019
New Revision: 270512
URL: https://gcc.gnu.org/viewcvs?rev=270512=gcc=rev
Log:
modulo-sched: prevent division by zero (PR87979)
PR
|--- |FIXED
Assignee|unassigned at gcc dot gnu.org |zhroma at gcc dot
gnu.org
--- Comment #6 from Roman Zhuykov ---
Fixed.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84032
--- Comment #5 from Roman Zhuykov ---
Author: zhroma
Date: Tue Apr 23 12:53:43 2019
New Revision: 270511
URL: https://gcc.gnu.org/viewcvs?rev=270511=gcc=rev
Log:
modulo-sched: fix branch scheduling issue (PR84032)
PR
64 matches
Mail list logo
