https://gcc.gnu.org/bugzilla/show_bug.cgi?id=110869
Xi Ruoyao changed:
What|Removed |Added
Resolution|--- |FIXED
CC|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=110869
Bug 110869 depends on bug 110867, which changed state.
Bug 110867 Summary: [14 Regression] ICE in combine after
7cdd0860949c6c3232e6cff1d7ca37bb5234074c
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=110867
What|Removed
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=110869
--- Comment #19 from CVS Commits ---
The master branch has been updated by Stefan Schulze Frielinghaus
:
https://gcc.gnu.org/g:f3a8dde08b420dbe9cacbc1a1febaaf77357f4d4
commit r14-3074-gf3a8dde08b420dbe9cacbc1a1febaaf77357f4d4
Author: Stefan Sc
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=110869
--- Comment #18 from Stefan Schulze Frielinghaus ---
Thanks again for testing. Very much appreciated!
I like the idea of a comment and posted a patch:
https://gcc.gnu.org/pipermail/gcc-patches/2023-August/626514.html
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=110869
--- Comment #17 from ro at CeBiTec dot Uni-Bielefeld.DE ---
> --- Comment #16 from Stefan Schulze Frielinghaus ibm.com> ---
> Turns out that my dejagnu foo is weak ;-) I came up with a wrong target
> selector. Should be fixed in the new attachm
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=110869
--- Comment #16 from Stefan Schulze Frielinghaus ---
Turns out that my dejagnu foo is weak ;-) I came up with a wrong target
selector. Should be fixed in the new attachment.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=110869
--- Comment #15 from Stefan Schulze Frielinghaus ---
Created attachment 55688
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=55688&action=edit
Increase optimization and skip sparc for 4-6
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=110869
--- Comment #14 from Stefan Schulze Frielinghaus ---
For -3 and -4 I can confirm that we do not end up with a proper comparison
during combine which means we should just ignore these on Sparc.
I'm currently puzzled that -5 and -6 are actually p
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=110869
--- Comment #13 from ro at CeBiTec dot Uni-Bielefeld.DE ---
> --- Comment #12 from Stefan Schulze Frielinghaus ibm.com> ---
> I have done a test with a cross-compiler and it looks to me as if we need -O2
> instead of -O1 on Sparc in order to tr
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=110869
--- Comment #12 from Stefan Schulze Frielinghaus ---
I have done a test with a cross-compiler and it looks to me as if we need -O2
instead of -O1 on Sparc in order to trigger the optimization. Can you give the
attached patch a try? Sorry for a
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=110869
--- Comment #11 from Stefan Schulze Frielinghaus ---
Created attachment 55686
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=55686&action=edit
Increase optimization
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=110869
--- Comment #10 from Rainer Orth ---
The tests still FAIL on Solaris/SPARC:
FAIL: gcc.dg/cmp-mem-const-1.c scan-rtl-dump combine "narrow comparison from
mode .I to QI"
FAIL: gcc.dg/cmp-mem-const-2.c scan-rtl-dump combine "narrow comparison from
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=110869
--- Comment #9 from Rainer Orth ---
Created attachment 55684
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=55684&action=edit
64-bit sparc-sun-solaris2.11 cmp-mem-const-1.c.289r.combine
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=110869
--- Comment #8 from CVS Commits ---
The master branch has been updated by Stefan Schulze Frielinghaus
:
https://gcc.gnu.org/g:41ef5a34161356817807be3a2e51fbdbe575ae85
commit r14-2932-g41ef5a34161356817807be3a2e51fbdbe575ae85
Author: Stefan Sch
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=110869
--- Comment #7 from Stefan Schulze Frielinghaus
---
I've send a patch for review:
https://gcc.gnu.org/pipermail/gcc-patches/2023-August/626075.html
and thanks for testing :)
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=110869
Eric Botcazou changed:
What|Removed |Added
Status|UNCONFIRMED |NEW
Ever confirmed|0
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=110869
Rainer Orth changed:
What|Removed |Added
CC||ebotcazou at gcc dot gnu.org
--- Comment
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=110869
--- Comment #4 from Stefan Schulze Frielinghaus
---
For sparc we already see some sort of pre-optimization which "breaks" the new
test cases. For example, for test cmp-mem-const-1.c we have prior combine:
(insn 14 13 41 2 (set (reg:SI 117)
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=110869
--- Comment #3 from ro at CeBiTec dot Uni-Bielefeld.DE ---
> --- Comment #2 from ro at CeBiTec dot Uni-Bielefeld.DE Uni-Bielefeld.DE> ---
>> --- Comment #1 from Andrew Pinski ---
>> Can you test the patch in bug 110867 comment #1 to see if fix
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=110869
--- Comment #2 from ro at CeBiTec dot Uni-Bielefeld.DE ---
> --- Comment #1 from Andrew Pinski ---
> Can you test the patch in bug 110867 comment #1 to see if fixes the issue here
> too?
Sure: sparc-sun-solaris2.11 bootstrap in progress...
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=110869
--- Comment #1 from Andrew Pinski ---
Can you test the patch in bug 110867 comment #1 to see if fixes the issue here
too?
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=110869
Rainer Orth changed:
What|Removed |Added
Target Milestone|--- |14.0
22 matches
Mail list logo