From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 79633 invoked by alias); 21 Mar 2016 19:06:09 -0000 Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Id: List-Archive: List-Post: List-Help: Sender: gcc-patches-owner@gcc.gnu.org Received: (qmail 79617 invoked by uid 89); 21 Mar 2016 19:06:08 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-1.9 required=5.0 tests=BAYES_00,RP_MATCHES_RCVD,SPF_HELO_PASS autolearn=ham version=3.3.2 spammy= X-HELO: mx1.redhat.com Received: from mx1.redhat.com (HELO mx1.redhat.com) (209.132.183.28) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with (AES256-GCM-SHA384 encrypted) ESMTPS; Mon, 21 Mar 2016 19:06:07 +0000 Received: from int-mx11.intmail.prod.int.phx2.redhat.com (int-mx11.intmail.prod.int.phx2.redhat.com [10.5.11.24]) by mx1.redhat.com (Postfix) with ESMTPS id 046E6461EA for ; Mon, 21 Mar 2016 19:06:05 +0000 (UTC) Received: from slagheap.utah.redhat.com (ovpn-113-52.phx2.redhat.com [10.3.113.52]) by int-mx11.intmail.prod.int.phx2.redhat.com (8.14.4/8.14.4) with ESMTP id u2LJ645M031712 for ; Mon, 21 Mar 2016 15:06:05 -0400 To: "gcc-patches@gcc.gnu.org >> gcc-patches" From: Jeff Law Subject: [RFA][PATCH] Adding missing calls to bitmap_clear Message-ID: <56F0461C.3070601@redhat.com> Date: Mon, 21 Mar 2016 19:10:00 -0000 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.6.0 MIME-Version: 1.0 Content-Type: multipart/mixed; boundary="------------070308090608020501070801" X-IsSubscribed: yes X-SW-Source: 2016-03/txt/msg01199.txt.bz2 This is a multi-part message in MIME format. --------------070308090608020501070801 Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit Content-length: 414 As noted last week, find_removable_extensions initializes several bitmaps, but doesn't clear them. This is not strictly a leak as the GC system should find dead data, but it's better to go ahead and clear the bitmaps. That releases the elements back to the cache and presumably makes things easier for the GC system as well. Bootstrapped and regression tested on x86_64-linux-gnu. OK for the trunk? Jeff --------------070308090608020501070801 Content-Type: text/plain; charset=UTF-8; name="P" Content-Transfer-Encoding: base64 Content-Disposition: inline; filename="P" Content-length: 509 CSogcmVlLmMgKGZpbmRfcmVtb3ZhYmxlX2V4dGVuc2lvbnMpOiBDbGVhciB0 aGUgbG9jYWwgYml0bWFwcy4KCmRpZmYgLS1naXQgYS9nY2MvcmVlLmMgYi9n Y2MvcmVlLmMKaW5kZXggMTNhN2EwNS4uM2RjMTgwYyAxMDA2NDQKLS0tIGEv Z2NjL3JlZS5jCisrKyBiL2djYy9yZWUuYwpAQCAtMTEzOSw2ICsxMTM5LDEw IEBAIGZpbmRfcmVtb3ZhYmxlX2V4dGVuc2lvbnMgKHZvaWQpCiAgICAgfQog CiAgIFhERUxFVEVWRUMgKGRlZl9tYXApOworICBiaXRtYXBfY2xlYXIgKCZp bml0KTsKKyAgYml0bWFwX2NsZWFyICgma2lsbCk7CisgIGJpdG1hcF9jbGVh ciAoJmdlbik7CisgIGJpdG1hcF9jbGVhciAoJnRtcCk7CiAKICAgcmV0dXJu IGluc25fbGlzdDsKIH0K --------------070308090608020501070801--