public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* [PATCH] Stop in note_eh_region_may_contain_throw after ERT_MUST_NOT_THROW (PR tree-optimization/48611)
@ 2011-04-18 21:48 Jakub Jelinek
  2011-04-20 18:25 ` Richard Henderson
  0 siblings, 1 reply; 2+ messages in thread
From: Jakub Jelinek @ 2011-04-18 21:48 UTC (permalink / raw)
  To: Richard Henderson; +Cc: gcc-patches

Hi!

My understanding is that RT_MUST_NOT_THROW regions shouldn't ever
propagate exceptions to outer regions, they'd terminate the program
instead.
So, we don't need to mark outer regions as possibly needing to handle
throw.

Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk?

2011-04-18  Jakub Jelinek  <jakub@redhat.com>

	PR tree-optimization/48611
	* tree-eh.c (note_eh_region_may_contain_throw): Don't propagate
	beyond ERT_MUST_NOT_THROW region.

--- gcc/tree-eh.c.jj	2011-01-25 18:40:08.000000000 +0100
+++ gcc/tree-eh.c	2011-04-18 15:28:17.000000000 +0200
@@ -849,6 +849,8 @@ note_eh_region_may_contain_throw (eh_reg
 {
   while (bitmap_set_bit (eh_region_may_contain_throw_map, region->index))
     {
+      if (region->type == ERT_MUST_NOT_THROW)
+	break;
       region = region->outer;
       if (region == NULL)
 	break;

	Jakub

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [PATCH] Stop in note_eh_region_may_contain_throw after ERT_MUST_NOT_THROW (PR tree-optimization/48611)
  2011-04-18 21:48 [PATCH] Stop in note_eh_region_may_contain_throw after ERT_MUST_NOT_THROW (PR tree-optimization/48611) Jakub Jelinek
@ 2011-04-20 18:25 ` Richard Henderson
  0 siblings, 0 replies; 2+ messages in thread
From: Richard Henderson @ 2011-04-20 18:25 UTC (permalink / raw)
  To: Jakub Jelinek; +Cc: gcc-patches

On 04/18/2011 02:35 PM, Jakub Jelinek wrote:
> 	PR tree-optimization/48611
> 	* tree-eh.c (note_eh_region_may_contain_throw): Don't propagate
> 	beyond ERT_MUST_NOT_THROW region.

Ok.


r~

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2011-04-20 18:00 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-04-18 21:48 [PATCH] Stop in note_eh_region_may_contain_throw after ERT_MUST_NOT_THROW (PR tree-optimization/48611) Jakub Jelinek
2011-04-20 18:25 ` Richard Henderson

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).