[Bug target/13158] bad unwind info as a result of sibcall
-- What|Removed |Added Status|SUSPENDED |NEW Last reconfirmed|2004-12-18 13:30:15 |2004-12-18 13:31:27 date|| http://gcc.gnu.org/bugzilla/show_bug.cgi?id=13158
[Bug target/13158] bad unwind info as a result of sibcall
--- Additional Comments From steven at gcc dot gnu dot org 2004-12-18 13:30 --- ...so I can reopen it. -- What|Removed |Added Last reconfirmed|2004-08-14 07:05:01 |2004-12-18 13:30:15 date|| http://gcc.gnu.org/bugzilla/show_bug.cgi?id=13158
[Bug target/13158] bad unwind info as a result of sibcall
--- Additional Comments From steven at gcc dot gnu dot org 2004-12-18 13:29 --- The patch to fix this introduces another wrong-code bug, PR18987. I have not checked if this bug is also present on the GCC 3.4 branch now. Suspending... -- What|Removed |Added Status|ASSIGNED|SUSPENDED http://gcc.gnu.org/bugzilla/show_bug.cgi?id=13158
[Bug target/13158] bad unwind info as a result of sibcall
--- Additional Comments From cvs-commit at gcc dot gnu dot org 2004-11-24 22:23 --- Subject: Bug 13158 CVSROOT:/cvs/gcc Module name:gcc Branch: gcc-3_4-rhl-branch Changes by: [EMAIL PROTECTED] 2004-11-24 22:22:29 Modified files: gcc: ChangeLog emit-rtl.c gcc/config/ia64: ia64.c Log message: 2004-10-27 David Mosberger <[EMAIL PROTECTED]> James E Wilson <[EMAIL PROTECTED]> PR target/13158 * config/ia64/ia64.c (ia64_expand_epilogue): Set RTX_FRAME_RELATED_P on sibcall alloc instruction. (process_set): Handle sibcall alloc instruction. 2004-10-26 James E Wilson <[EMAIL PROTECTED]> PR target/18010 * emit-rtl.c (emit_copy_of_insn_after): Copy RTX_FRAME_RELATED_P. 2004-02-03 Kazu Hirata <[EMAIL PROTECTED]> * config/ia64/ia64.c: Use const0_rtx instead of GEN_INT (0). Patches: http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/ChangeLog.diff?cvsroot=gcc&only_with_tag=gcc-3_4-rhl-branch&r1=2.2326.2.399.2.57&r2=2.2326.2.399.2.58 http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/emit-rtl.c.diff?cvsroot=gcc&only_with_tag=gcc-3_4-rhl-branch&r1=1.365.4.5&r2=1.365.4.5.2.1 http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/config/ia64/ia64.c.diff?cvsroot=gcc&only_with_tag=gcc-3_4-rhl-branch&r1=1.265.2.6.2.5&r2=1.265.2.6.2.6 -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=13158
[Bug target/13158] bad unwind info as a result of sibcall
--- Additional Comments From davidm at hpl dot hp dot com 2004-10-28 09:27 --- (In reply to comment #10) > I've now checked the patch into mainline. Thanks! > Adding the patch to gcc-3.4 requires that it be a regression. This doesn't > seem to qualify according to a strict interpretation of the rules. I'm not in tune with GCC release management so you/the release-manager will have to make the call. However, if we can't put the unwind fix(es) into 3.4, it will mean we have to ask each distributor to pick them up separately. This is a lot of work for something that is (a) guaranteed to be ia64-specific (i.e., the patches won't affect any other platforms) and (b) is known to cause real problems. Just my 2 cents... -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=13158
[Bug target/13158] bad unwind info as a result of sibcall
--- Additional Comments From wilson at gcc dot gnu dot org 2004-10-27 23:16 --- Thanks for the help. I got bogged down trying to figure out how to debug gcj/libjava problems, and wasn't able to make any useful progress before getting distracted by other stuff. I've now checked the patch into mainline. Adding the patch to gcc-3.4 requires that it be a regression. This doesn't seem to qualify according to a strict interpretation of the rules. The same broken code is in gcc-3.0.4, and gcc-2.96 doesn't really count, as there was no official FSF gcc release including the IA-64 support before gcc-3.x. -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=13158
[Bug target/13158] bad unwind info as a result of sibcall
--- Additional Comments From cvs-commit at gcc dot gnu dot org 2004-10-27 22:44 --- Subject: Bug 13158 CVSROOT:/cvs/gcc Module name:gcc Changes by: [EMAIL PROTECTED] 2004-10-27 22:44:09 Modified files: gcc: ChangeLog gcc/config/ia64: ia64.c Log message: IA-64 sibcall unwind bug, PR target/13158 * config/ia64/ia64.c (ia64_expand_epilogue): Set RTX_FRAME_RELATED_P on sibcall alloc instruction. (process_set): Handle sibcall alloc instruction. Patches: http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/ChangeLog.diff?cvsroot=gcc&r1=2.6057&r2=2.6058 http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/config/ia64/ia64.c.diff?cvsroot=gcc&r1=1.328&r2=1.329 -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=13158
[Bug target/13158] bad unwind info as a result of sibcall
--- Additional Comments From davidm at hpl dot hp dot com 2004-10-22 16:25 --- (In reply to comment #6) > Subject: Re: bad unwind info as a result of sibcall > > I tried testing the patch, and ended up with 2 extra libjava failures. I was able to reproduce the libjava failures. The problem was that the patch didn't do the full epilogue processing for the sibcall. With the "revised-patch" applied, I get no additional failures anymore. However, due to a gas bug it produces epilogue directives with "t" value of -1 which isn't good (though it doesn't hurt the existing ia64 linux unwinders). I'll submit a patch for gas to fix this problem. If there are no other issues, could this patch be checked in? -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=13158
