public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* [PATCH, SMS] Avoid unfreed memory when SMS fails
@ 2011-04-15 15:27 Revital Eres
  2011-04-15 15:42 ` Richard Guenther
  0 siblings, 1 reply; 2+ messages in thread
From: Revital Eres @ 2011-04-15 15:27 UTC (permalink / raw)
  To: zaks; +Cc: Patch Tracking, gcc-patches

Hello,

This patch fixes the scenario where SMS fails to
schedule a loop and continue to the next one without
freeing data structures allocated while scheduling
the first loop.

Bootstrap and regtested on ppc64-redhat-linux.

OK for mainline?

Thanks,
Revital

Changelog:

* modulo-sched.c (sms_schedule): Avoid unfreed memory when SMS fails.

Index: modulo-sched.c
===================================================================
--- modulo-sched.c      (revision 170464)
+++ modulo-sched.c      (working copy)
@@ -1177,7 +1177,6 @@ sms_schedule (void)
              fprintf (dump_file, HOST_WIDEST_INT_PRINT_DEC, trip_count);
              fprintf (dump_file, ")\n");
            }
-         continue;
        }
       else
        {

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

* Re: [PATCH, SMS] Avoid unfreed memory when SMS fails
  2011-04-15 15:27 [PATCH, SMS] Avoid unfreed memory when SMS fails Revital Eres
@ 2011-04-15 15:42 ` Richard Guenther
  0 siblings, 0 replies; 2+ messages in thread
From: Richard Guenther @ 2011-04-15 15:42 UTC (permalink / raw)
  To: Revital Eres; +Cc: zaks, Patch Tracking, gcc-patches

On Fri, Apr 15, 2011 at 5:26 PM, Revital Eres <revital.eres@linaro.org> wrote:
> Hello,
>
> This patch fixes the scenario where SMS fails to
> schedule a loop and continue to the next one without
> freeing data structures allocated while scheduling
> the first loop.
>
> Bootstrap and regtested on ppc64-redhat-linux.
>
> OK for mainline?

Ok.

Thanks,
Richard.

> Thanks,
> Revital
>
> Changelog:
>
> * modulo-sched.c (sms_schedule): Avoid unfreed memory when SMS fails.
>
> Index: modulo-sched.c
> ===================================================================
> --- modulo-sched.c      (revision 170464)
> +++ modulo-sched.c      (working copy)
> @@ -1177,7 +1177,6 @@ sms_schedule (void)
>              fprintf (dump_file, HOST_WIDEST_INT_PRINT_DEC, trip_count);
>              fprintf (dump_file, ")\n");
>            }
> -         continue;
>        }
>       else
>        {
>

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

end of thread, other threads:[~2011-04-15 15:40 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-04-15 15:27 [PATCH, SMS] Avoid unfreed memory when SMS fails Revital Eres
2011-04-15 15:42 ` Richard Guenther

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).