From patchwork Thu Sep 30 18:01:32 2010 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Sebastian Pop X-Patchwork-Id: 66226 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from sourceware.org (server1.sourceware.org [209.132.180.131]) by ozlabs.org (Postfix) with SMTP id 807B9B70A3 for ; Fri, 1 Oct 2010 04:08:05 +1000 (EST) Received: (qmail 21693 invoked by alias); 30 Sep 2010 18:06:04 -0000 Received: (qmail 13541 invoked by uid 22791); 30 Sep 2010 18:04:05 -0000 X-SWARE-Spam-Status: No, hits=-1.9 required=5.0 tests=AWL, BAYES_00, DKIM_SIGNED, DKIM_VALID, DKIM_VALID_AU, FREEMAIL_FROM, RCVD_IN_DNSWL_NONE, TW_SV, T_TO_NO_BRKTS_FREEMAIL X-Spam-Check-By: sourceware.org Received: from mail-gy0-f175.google.com (HELO mail-gy0-f175.google.com) (209.85.160.175) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Thu, 30 Sep 2010 18:03:55 +0000 Received: by mail-gy0-f175.google.com with SMTP id 8so110434gyg.20 for ; Thu, 30 Sep 2010 11:03:54 -0700 (PDT) Received: by 10.101.107.8 with SMTP id j8mr4635828anm.166.1285869834854; Thu, 30 Sep 2010 11:03:54 -0700 (PDT) Received: from napoca ([163.181.251.115]) by mx.google.com with ESMTPS id f22sm161636anh.4.2010.09.30.11.03.52 (version=TLSv1/SSLv3 cipher=RC4-MD5); Thu, 30 Sep 2010 11:03:54 -0700 (PDT) Received: by napoca (sSMTP sendmail emulation); Thu, 30 Sep 2010 13:03:51 -0500 From: Sebastian Pop To: gcc-patches@gcc.gnu.org Cc: gcc-graphite@googlegroups.com Subject: [PATCH 40/44] Fix Graphite memory leaks. Date: Thu, 30 Sep 2010 13:01:32 -0500 Message-Id: <1285869696-10915-41-git-send-email-sebpop@gmail.com> In-Reply-To: <1285869696-10915-1-git-send-email-sebpop@gmail.com> References: <1285869696-10915-1-git-send-email-sebpop@gmail.com> X-IsSubscribed: yes Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Id: List-Unsubscribe: List-Archive: List-Post: List-Help: Sender: gcc-patches-owner@gcc.gnu.org Delivered-To: mailing list gcc-patches@gcc.gnu.org From: spop 2010-09-23 Sebastian Pop * graphite-clast-to-gimple.c (compute_bounds_for_level): Free le and ps. * graphite-poly.c (pbb_number_of_iterations_at_time): Free le and domain. * graphite-sese-to-poly.c (add_upper_bounds_from_estimated_nit): Do not allocate ub_expr, it is passed in initialized. git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/branches/graphite@164565 138bc75d-0d04-0410-961f-82ee72b054a4 --- gcc/ChangeLog | 9 +++++++++ gcc/ChangeLog.graphite | 9 +++++++++ gcc/graphite-clast-to-gimple.c | 2 ++ gcc/graphite-poly.c | 2 ++ gcc/graphite-sese-to-poly.c | 1 - 5 files changed, 22 insertions(+), 1 deletions(-) diff --git a/gcc/ChangeLog b/gcc/ChangeLog index a742e33..3e87861 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,12 @@ +2010-09-30 Sebastian Pop + + * graphite-clast-to-gimple.c (compute_bounds_for_level): Free le + and ps. + * graphite-poly.c (pbb_number_of_iterations_at_time): Free le and + domain. + * graphite-sese-to-poly.c (add_upper_bounds_from_estimated_nit): + Do not allocate ub_expr, it is passed in initialized. + 2010-09-30 Andreas Simbuerger * graphite-clast-to-gimple.c (init_cloog_input_file): New. diff --git a/gcc/ChangeLog.graphite b/gcc/ChangeLog.graphite index ba7d99e..1dddf4c 100644 --- a/gcc/ChangeLog.graphite +++ b/gcc/ChangeLog.graphite @@ -1,3 +1,12 @@ +2010-09-23 Sebastian Pop + + * graphite-clast-to-gimple.c (compute_bounds_for_level): Free le + and ps. + * graphite-poly.c (pbb_number_of_iterations_at_time): Free le and + domain. + * graphite-sese-to-poly.c (add_upper_bounds_from_estimated_nit): + Do not allocate ub_expr, it is passed in initialized. + 2010-09-21 Andreas Simbuerger * common.opt (fgraphite-dump-cloog): New flag. diff --git a/gcc/graphite-clast-to-gimple.c b/gcc/graphite-clast-to-gimple.c index faefbb6..9a90ef7 100644 --- a/gcc/graphite-clast-to-gimple.c +++ b/gcc/graphite-clast-to-gimple.c @@ -703,6 +703,8 @@ compute_bounds_for_level (poly_bb_p pbb, int level, mpz_t low, mpz_t up) ppl_max_for_le_pointset (ps, le, up); ppl_min_for_le_pointset (ps, le, low); + ppl_delete_Linear_Expression (le); + ppl_delete_Pointset_Powerset_C_Polyhedron (ps); } /* Compute the type for the induction variable at LEVEL for the diff --git a/gcc/graphite-poly.c b/gcc/graphite-poly.c index 1644ecf..9f67e85 100644 --- a/gcc/graphite-poly.c +++ b/gcc/graphite-poly.c @@ -1738,8 +1738,10 @@ pbb_number_of_iterations_at_time (poly_bb_p pbb, mpz_clear (diff); mpz_clear (lb); mpz_clear (ub); + ppl_delete_Linear_Expression (le); ppl_delete_Pointset_Powerset_C_Polyhedron (sctr_ub); ppl_delete_Pointset_Powerset_C_Polyhedron (sctr_lb); + ppl_delete_Pointset_Powerset_C_Polyhedron (domain); } /* Translates LOOP to LST. */ diff --git a/gcc/graphite-sese-to-poly.c b/gcc/graphite-sese-to-poly.c index 1ee6fee..87b226b 100644 --- a/gcc/graphite-sese-to-poly.c +++ b/gcc/graphite-sese-to-poly.c @@ -1032,7 +1032,6 @@ add_upper_bounds_from_estimated_nit (scop_p scop, double_int nit, ppl_Coefficient_t coef; ppl_Constraint_t ub; - ppl_new_Linear_Expression_with_dimension (&ub_expr, dim); ppl_new_C_Polyhedron_from_space_dimension (&pol, dim, 0); ppl_new_Linear_Expression_from_Linear_Expression (&nb_iters_le, ub_expr);