[Bug target/56858] alpha looks for NOTE_INSN_EH_REGION notes that cannot exist

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=56858 Uroš Bizjak ubizjak at gmail dot com changed: What|Removed |Added Status|NEW |RESOLVED

[Bug target/56858] alpha looks for NOTE_INSN_EH_REGION notes that cannot exist

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=56858 --- Comment #16 from uros at gcc dot gnu.org --- Author: uros Date: Fri Jul 11 07:42:50 2014 New Revision: 212449 URL: https://gcc.gnu.org/viewcvs?rev=212449root=gccview=rev Log: PR target/56858 * config/alpha/alpha.c: Include

[Bug target/56858] alpha looks for NOTE_INSN_EH_REGION notes that cannot exist

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=56858 --- Comment #14 from Uroš Bizjak ubizjak at gmail dot com --- (In reply to Uroš Bizjak from comment #12) Steven, is it possible to emit NOTE_INSN_EH_REGION_END in such way that it would not split the call and its NOTE_INSN_CALL_ARG_LOCATION?

[Bug target/56858] alpha looks for NOTE_INSN_EH_REGION notes that cannot exist

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=56858 --- Comment #15 from Uroš Bizjak ubizjak at gmail dot com --- Created attachment 33009 -- https://gcc.gnu.org/bugzilla/attachment.cgi?id=33009action=edit Updated patch that introduces trap_shadows pass after eh_ranges Updated patch due to the

[Bug target/56858] alpha looks for NOTE_INSN_EH_REGION notes that cannot exist

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=56858 --- Comment #12 from Uros Bizjak ubizjak at gmail dot com 2013-04-21 21:15:50 UTC --- Created attachment 29911 -- http://gcc.gnu.org/bugzilla/attachment.cgi?id=29911 Patch that introduces trap_shadows pass after eh_ranges 2013-04-18

[Bug target/56858] alpha looks for NOTE_INSN_EH_REGION notes that cannot exist

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=56858 --- Comment #13 from stevenb.gcc at gmail dot com stevenb.gcc at gmail dot com 2013-04-21 21:36:32 UTC --- Steven, is it possible to emit NOTE_INSN_EH_REGION_END in such way that it would not split the call and its

[Bug target/56858] alpha looks for NOTE_INSN_EH_REGION notes that cannot exist

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=56858 --- Comment #4 from Uros Bizjak ubizjak at gmail dot com 2013-04-08 07:38:26 UTC --- (In reply to comment #2) I can't test this because I have no access to alpha hardware. The testsuite results [1] are OK. [1]

[Bug target/56858] alpha looks for NOTE_INSN_EH_REGION notes that cannot exist

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=56858 --- Comment #5 from Richard Henderson rth at gcc dot gnu.org 2013-04-08 14:35:03 UTC --- They did used to exist during mach_reorg; they used to exist at rtl creation time. You cannot simply delete the code either, that produces wrong-code

[Bug target/56858] alpha looks for NOTE_INSN_EH_REGION notes that cannot exist

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=56858 --- Comment #6 from Steven Bosscher steven at gcc dot gnu.org 2013-04-08 15:32:47 UTC --- (In reply to comment #5) They did used to exist during mach_reorg; they used to exist at rtl creation time. You cannot simply delete the code

[Bug target/56858] alpha looks for NOTE_INSN_EH_REGION notes that cannot exist

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=56858 --- Comment #7 from Richard Henderson rth at gcc dot gnu.org 2013-04-08 15:52:13 UTC --- (In reply to comment #6) Do you recall when that was? Must have been long, long ago... I've tried to find where it happened, and the rewrite was

[Bug target/56858] alpha looks for NOTE_INSN_EH_REGION notes that cannot exist

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=56858 --- Comment #8 from Steven Bosscher steven at gcc dot gnu.org 2013-04-08 18:43:25 UTC --- (In reply to comment #7) (In reply to comment #6) Do you recall when that was? Must have been long, long ago... I've tried to find where it

[Bug target/56858] alpha looks for NOTE_INSN_EH_REGION notes that cannot exist

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=56858 --- Comment #9 from Richard Henderson rth at gcc dot gnu.org 2013-04-08 19:27:03 UTC --- (In reply to comment #8) The part of alpha EH that depends on the notes must have been been broken since than, more than 12 years. Without anyone

[Bug target/56858] alpha looks for NOTE_INSN_EH_REGION notes that cannot exist

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=56858 --- Comment #10 from stevenb.gcc at gmail dot com stevenb.gcc at gmail dot com 2013-04-08 19:42:51 UTC --- Sure. Do you know any package that combines C++ EH with IEEE exceptions? I don't. In fact, I don't actually know of a package

[Bug target/56858] alpha looks for NOTE_INSN_EH_REGION notes that cannot exist

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=56858 --- Comment #11 from Richard Henderson rth at gcc dot gnu.org 2013-04-08 20:09:47 UTC --- (In reply to comment #10) Removing the notes entirely is the goal eventually, but I'd settle for making them local to final.c for now. Hmm.

[Bug target/56858] alpha looks for NOTE_INSN_EH_REGION notes that cannot exist

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=56858 --- Comment #3 from Uros Bizjak ubizjak at gmail dot com 2013-04-07 08:28:44 UTC --- (In reply to comment #2) I can't test this because I have no access to alpha hardware. I'll bootstrap and regression test the patch on ev68, please

[Bug target/56858] alpha looks for NOTE_INSN_EH_REGION notes that cannot exist

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=56858 Steven Bosscher steven at gcc dot gnu.org changed: What|Removed |Added Target|

[Bug target/56858] alpha looks for NOTE_INSN_EH_REGION notes that cannot exist

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=56858 --- Comment #2 from Steven Bosscher steven at gcc dot gnu.org 2013-04-06 21:55:57 UTC --- Proposed fix: Index: config/alpha/alpha.c === --- config/alpha/alpha.c