[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 changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

[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=212449&root=gcc&view=rev Log: PR target/56858 * config/alpha/alpha.c: Include tree-pass

[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 --- Created attachment 33009 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=33009&action=edit Updated patch that introduces trap_shadows pass after eh_ranges Updated patch due to the new pass manager.

[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 --- (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? This would > solve the a

[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 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 NOTE_INSN_CALL_ARG_LOCATION? This would solve > th

[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 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 Uros Bizjak PR

[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 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. Well, assuming we want

[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 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 that wants to enable > trappin

[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 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 noticing... Sure.

[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 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 happened, and the rewrit

[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 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 done before > at least

[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 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 either, that produces > wr

[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 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 wrt fp exceptions. Of

[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 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] http://gcc.gnu.org/ml/gcc-testresults/2013-04

[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 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 expect results in a coup

[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 2013-04-06 21:55:57 UTC --- Proposed fix: Index: config/alpha/alpha.c === --- config/alpha/alpha.c(revision 197532) +++ config/

[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 changed: What|Removed |Added Target||alpha-*-* Status|U