From patchwork Thu Oct 24 17:53:08 2013 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Sriraman Tallam X-Patchwork-Id: 285993 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]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (Client did not present a certificate) by ozlabs.org (Postfix) with ESMTPS id A0CF32C00C8 for ; Fri, 25 Oct 2013 04:53:18 +1100 (EST) DomainKey-Signature: a=rsa-sha1; c=nofws; d=gcc.gnu.org; h=list-id :list-unsubscribe:list-archive:list-post:list-help:sender :mime-version:date:message-id:subject:from:to:content-type; q= dns; s=default; b=PvoKst6DkYfYJpnAvLbfFRYpxBiz1oLlXNS/L8bRdYtfZx O2sfmyVlu5FkJH8F9w+7/90lbTOnoKuv7tkINxjA4VKyNDe27lRqs30fOIMwBd2N aBDVEOEcxLL9D+weu8DlJ7VaeXZUrrfP0Ay559QJU11LnrhnYQfekY5R0duC4= DKIM-Signature: v=1; a=rsa-sha1; c=relaxed; d=gcc.gnu.org; h=list-id :list-unsubscribe:list-archive:list-post:list-help:sender :mime-version:date:message-id:subject:from:to:content-type; s= default; bh=EdMniT1JHWZLxkzPSXEQuCCFnbI=; b=bIQPcY6B0yOJcS9apoWG gdCmyZyow7g7sgUla++tSiVkLik4EyIELvT9Y6bxJJfzg7hqkaFCvVmCfaYAHYlj wj47epXSH5COiFFYKmSb19b1ccRSG+/IU/1EN0UZxN/RdM6H+kANs5dQtvO4LA0W 34CXcrVPuwQKZZiAkq8Tmf4= Received: (qmail 14546 invoked by alias); 24 Oct 2013 17:53:12 -0000 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 Received: (qmail 14530 invoked by uid 89); 24 Oct 2013 17:53:11 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-3.3 required=5.0 tests=AWL, BAYES_00, RCVD_IN_DNSWL_LOW, RP_MATCHES_RCVD, SPF_PASS autolearn=ham version=3.3.2 X-HELO: mail-ve0-f174.google.com Received: from mail-ve0-f174.google.com (HELO mail-ve0-f174.google.com) (209.85.128.174) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with (AES128-SHA encrypted) ESMTPS; Thu, 24 Oct 2013 17:53:10 +0000 Received: by mail-ve0-f174.google.com with SMTP id pa12so1158397veb.5 for ; Thu, 24 Oct 2013 10:53:08 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:mime-version:date:message-id:subject:from:to :content-type; bh=ofclAAjuyu65OhWLExAcTHueanfPHX4i1NX0ZNt8I8M=; b=Y/yTD95uWOb97plVZghNyHdReS71lAVxk+5tnKqDQQYpnbtgAdTHNvFJHVAvX8T6N3 GVGZqXC1ONEDss8A3Juk4N0LiUcdElzleCrYYXTzHhDbH4cO+lgS052Yxcx1nxDR6vET tjttmDHQRdkqPxdX8czWMd9z5HvFeAFhdT0gisEMqFYiLQj03547zuE5WysnuTR6ZJ0+ 4GOaTMBShjkh88QNaCQDmAKPRTClWXrNYow7OixCIyTqUD7WuCGym60/I7BjIsjPrdJM A1j12rV/rxijcw+GbleqfGIimgNw7CwieNoyECl+Aj1GujLuTOAg3KAgc1Q4UB7ts+pn kh1A== X-Gm-Message-State: ALoCoQlC2+cmLmRtU+EUrOYfkHACWefOQXsIgQtIan027zo4nQsui2cEpIICa8Ha7do0KiUhNCBIVHC9osjzEGoVY74P4RlTydlhOP7Asu6GHjwCcZinvDFJz3xiL2uMq4C2+afMzZSlawcYV4/QeC+umxMmy04ct4IfhtYZBW6xR7pryP1xm5IBoW/khTVWLKjZS7I0Q0XRwgT8xHHkIBqCvwd/O8JzCw== MIME-Version: 1.0 X-Received: by 10.58.238.9 with SMTP id vg9mr1382524vec.43.1382637188350; Thu, 24 Oct 2013 10:53:08 -0700 (PDT) Received: by 10.52.167.35 with HTTP; Thu, 24 Oct 2013 10:53:08 -0700 (PDT) Date: Thu, 24 Oct 2013 10:53:08 -0700 Message-ID: Subject: [patch] Patch to fix garbage collection problem with cgraph_fnver_htab From: Sriraman Tallam To: GCC Patches , Jan Hubicka , David Li X-IsSubscribed: yes Hi Honza, It looks like cgraph_fnver_htab defined in cgraph.c is not added to gc root in gt-cgraph.h. This patch fixes it. * cgraph.c (cgraph_fnver_htab): Move GTY((...)) to be before htab_t. Change param_is to use the struct and not the pointer to the struct. Ok to submit? Thanks Sri Index: gcc/cgraph.c =================================================================== --- gcc/cgraph.c (revision 204022) +++ gcc/cgraph.c (working copy) @@ -138,7 +138,7 @@ bool cpp_implicit_aliases_done; The cgraph_function_version_info has a THIS_NODE field that is the corresponding cgraph_node.. */ -static htab_t GTY((param_is (struct cgraph_function_version_info *))) +static GTY((param_is (struct cgraph_function_version_info))) htab_t cgraph_fnver_htab = NULL; /* Hash function for cgraph_fnver_htab. */