[Bug bootstrap/83015] [8 regression] bootstrap comparison failure on ia64
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83015 Jan Hubicka changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|--- |FIXED --- Comment #20 from Jan Hubicka --- Fixed.
[Bug bootstrap/83015] [8 regression] bootstrap comparison failure on ia64
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83015 --- Comment #19 from Jan Hubicka --- Author: hubicka Date: Fri Nov 24 11:24:55 2017 New Revision: 255138 URL: https://gcc.gnu.org/viewcvs?rev=255138&root=gcc&view=rev Log: PR bootstrap/83015 * ipa-inline.c (inline_small_functions): Set current badnes correctly when skipping checking. Modified: trunk/gcc/ChangeLog trunk/gcc/ipa-inline.c
[Bug bootstrap/83015] [8 regression] bootstrap comparison failure on ia64
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83015 --- Comment #18 from Jan Hubicka --- With the release checking in stage1 it reproduces on x86-64, too. I am testing Index: ipa-inline.c === --- ipa-inline.c(revision 255103) +++ ipa-inline.c(working copy) @@ -1865,6 +1865,8 @@ inline_small_functions (void) gcc_assert (cached_badness == current_badness); gcc_assert (current_badness >= badness); } + else +current_badness = edge_badness (edge, false); #else current_badness = edge_badness (edge, false); #endif
[Bug bootstrap/83015] [8 regression] bootstrap comparison failure on ia64
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83015 --- Comment #17 from Jan Hubicka --- Indeed with the configure flags used by your tester it does reproduce. It is difference in inlining priority order, perhaps caused by the badness sanity check (though I fail to see how)
[Bug bootstrap/83015] [8 regression] bootstrap comparison failure on ia64
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83015 --- Comment #16 from Jan Hubicka --- ytrium also bootstraps for me. I see you use release checking for stage1 that may be the trigger, I will try it now.
[Bug bootstrap/83015] [8 regression] bootstrap comparison failure on ia64
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83015 Jan Hubicka changed: What|Removed |Added Status|NEW |ASSIGNED Assignee|unassigned at gcc dot gnu.org |hubicka at gcc dot gnu.org --- Comment #15 from Jan Hubicka --- I believe it was done before the inliner change, but i am now trying to reproduce it on yttrium. It is bad idea to use periodic benchmarking machine for random bootstraps anyway. Thanks for the info and pointer to working itanium box!
[Bug bootstrap/83015] [8 regression] bootstrap comparison failure on ia64
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83015 --- Comment #14 from Andreas Schwab --- (In reply to Jan Hubicka from comment #11) > I was able to bootstrap on ia64 box (terbium) shortly before the unwind > changes went in without the bootstrap miscompare. Your testing was invalidated by the broken inline.
[Bug bootstrap/83015] [8 regression] bootstrap comparison failure on ia64
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83015 --- Comment #13 from Andreas Schwab --- e0fbd2cc7d2a77e5d06bdb7bf92b98a03b9b09ce is the first bad commit commit e0fbd2cc7d2a77e5d06bdb7bf92b98a03b9b09ce Author: hubicka Date: Mon Nov 20 09:55:02 2017 + PR bootstrap/83062 * ipa-inline.c (can_inline_edge_p): Fix typo in previous patch. git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/trunk@254946 138bc75d-0d04-0410-961f-82ee72b054a4
[Bug bootstrap/83015] [8 regression] bootstrap comparison failure on ia64
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83015 --- Comment #12 from Andreas Schwab --- See yttrium:/usr/local/gcc.
[Bug bootstrap/83015] [8 regression] bootstrap comparison failure on ia64
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83015 --- Comment #11 from Jan Hubicka --- I was able to bootstrap on ia64 box (terbium) shortly before the unwind changes went in without the bootstrap miscompare. So does the miscompare still reproduce for you and if so, would be posible to attach dumps?
[Bug bootstrap/83015] [8 regression] bootstrap comparison failure on ia64
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83015 --- Comment #10 from Andreas Schwab --- See comment#0.
[Bug bootstrap/83015] [8 regression] bootstrap comparison failure on ia64
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83015 --- Comment #9 from igor.v.tsimbalist at intel dot com --- I do not have an ia64 system at hand to reproduce. Could you please send me the log file or at least the part with error? Is the error still the same or is it different error? Or could you please update the bug with how it fails now? Igor > -Original Message- > From: sch...@linux-m68k.org [mailto:gcc-bugzi...@gcc.gnu.org] > Sent: Tuesday, November 21, 2017 10:24 AM > To: Tsimbalist, Igor V > Subject: [Bug bootstrap/83015] [8 regression] bootstrap comparison failure > on ia64 > > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83015 > > Andreas Schwab changed: > >What|Removed |Added > > Status|WAITING |NEW > > --- Comment #8 from Andreas Schwab --- > Still broken. > > -- > You are receiving this mail because: > You are on the CC list for the bug.
[Bug bootstrap/83015] [8 regression] bootstrap comparison failure on ia64
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83015 Andreas Schwab changed: What|Removed |Added Status|WAITING |NEW --- Comment #8 from Andreas Schwab --- Still broken.
[Bug bootstrap/83015] [8 regression] bootstrap comparison failure on ia64
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83015 --- Comment #7 from itsimbal at gcc dot gnu.org --- Author: itsimbal Date: Mon Nov 20 12:30:25 2017 New Revision: 254951 URL: https://gcc.gnu.org/viewcvs?rev=254951&root=gcc&view=rev Log: PR bootstrap/83015 * config/cr16/unwind-cr16.c (uw_install_context): Add FRAMES parameter. * config/xtensa/unwind-dw2-xtensa.c: Likewise * config/ia64/unwind-ia64.c: Add frames parameter. * unwind-sjlj.c: Likewise. Modified: trunk/libgcc/ChangeLog trunk/libgcc/config/cr16/unwind-cr16.c trunk/libgcc/config/ia64/unwind-ia64.c trunk/libgcc/config/xtensa/unwind-dw2-xtensa.c trunk/libgcc/unwind-sjlj.c
[Bug bootstrap/83015] [8 regression] bootstrap comparison failure on ia64
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83015 --- Comment #6 from igor.v.tsimbalist at intel dot com --- Andreas has sent this issue as a reply to my commit. I proposed a fix and asked for approval. Here is my reply https://gcc.gnu.org/ml/gcc-patches/2017-11/msg01647.html I have attached the patch also.
[Bug bootstrap/83015] [8 regression] bootstrap comparison failure on ia64
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83015 --- Comment #5 from igor.v.tsimbalist at intel dot com --- Created attachment 42658 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=42658&action=edit patch
[Bug bootstrap/83015] [8 regression] bootstrap comparison failure on ia64
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83015 Jan Hubicka changed: What|Removed |Added CC||igor.v.tsimbalist at intel dot com --- Comment #4 from Jan Hubicka --- Igor, the unwind issue seems to be yours.
[Bug bootstrap/83015] [8 regression] bootstrap comparison failure on ia64
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83015 --- Comment #3 from Jan Hubicka --- Terbium however now fails with In file included from /gcc/spec/sb-terbium-head-64/gcc/libgcc/config/ia64/unwind-ia64.c:2448: /gcc/spec/sb-terbium-head-64/gcc/libgcc/unwind.inc: In function '_Unwind_Resume_or_Rethrow': /gcc/spec/sb-terbium-head-64/gcc/libgcc/unwind.inc:273:3: error: too many arguments to function 'uw_install_context' uw_install_context (&this_context, &cur_context, frames); ^~ /gcc/spec/sb-terbium-head-64/gcc/libgcc/config/ia64/unwind-ia64.c:2167:1: note: declared here uw_install_context (struct _Unwind_Context *current __attribute__((unused)),
[Bug bootstrap/83015] [8 regression] bootstrap comparison failure on ia64
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83015 Jan Hubicka changed: What|Removed |Added Status|NEW |WAITING --- Comment #2 from Jan Hubicka --- I was able to reproduce it on terbium but it bootstraps fine now. Is the problem fixed for you? If not, would it be possible to have -fdump-tree-all-details-blocks -fdump-ipa-all-details for mismatching object file?
[Bug bootstrap/83015] [8 regression] bootstrap comparison failure on ia64
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83015 Richard Biener changed: What|Removed |Added Target Milestone|--- |8.0
[Bug bootstrap/83015] [8 regression] bootstrap comparison failure on ia64
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83015 Jan Hubicka changed: What|Removed |Added Status|UNCONFIRMED |NEW Last reconfirmed||2017-11-16 CC||hubicka at gcc dot gnu.org Ever confirmed|0 |1 --- Comment #1 from Jan Hubicka --- I am trying to reproduce it - so far I have learnt that compile farm computer listed as itanium is not, so bootstrap passed there.