From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 17670 invoked by alias); 8 Nov 2007 02:20:42 -0000 Received: (qmail 17660 invoked by uid 22791); 8 Nov 2007 02:20:41 -0000 X-Spam-Check-By: sourceware.org Received: from smtp-out.google.com (HELO smtp-out.google.com) (216.239.45.13) by sourceware.org (qpsmtpd/0.31) with ESMTP; Thu, 08 Nov 2007 02:20:38 +0000 Received: from zps37.corp.google.com (zps37.corp.google.com [172.25.146.37]) by smtp-out.google.com with ESMTP id lA82KXDM022461 for ; Wed, 7 Nov 2007 18:20:33 -0800 Received: from el-out-1112.google.com (eles27.prod.google.com [10.126.170.27]) by zps37.corp.google.com with ESMTP id lA82KW1x014683 for ; Wed, 7 Nov 2007 18:20:33 -0800 Received: by el-out-1112.google.com with SMTP id s27so7539ele for ; Wed, 07 Nov 2007 18:20:32 -0800 (PST) Received: by 10.78.138.14 with SMTP id l14mr34163hud.1194488432546; Wed, 07 Nov 2007 18:20:32 -0800 (PST) Received: by 10.78.43.13 with HTTP; Wed, 7 Nov 2007 18:20:32 -0800 (PST) Message-ID: Date: Thu, 08 Nov 2007 02:20:00 -0000 From: "Diego Novillo" To: "Jakub Jelinek" Subject: Re: Fix PR 33870 Cc: gcc-patches@gcc.gnu.org In-Reply-To: <20071108003507.GQ5451@devserv.devel.redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Content-Disposition: inline References: <20071108003507.GQ5451@devserv.devel.redhat.com> X-IsSubscribed: yes 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 X-SW-Source: 2007-11/txt/msg00378.txt.bz2 On Nov 7, 2007 7:35 PM, Jakub Jelinek wrote: > Doesn't this grow struct tree_struct_field_tag by in most cases 8 bytes? Oops, silly me. Thanks for noticing. Fixed with: * tree.h (struct tree_struct_field_tag): Move field in_nested_struct ... (struct tree_memory_tag): ... here. Index: tree.h =================================================================== --- tree.h (revision 129976) +++ tree.h (working copy) @@ -2554,7 +2554,11 @@ struct tree_memory_tag GTY(()) bitmap GTY ((skip)) aliases; + /* True if this tag has global scope. */ unsigned int is_global:1; + + /* True if this SFT is for a field in a nested structure. */ + unsigned int in_nested_struct : 1; }; #define MTAG_GLOBAL(NODE) (TREE_MEMORY_TAG_CHECK (NODE)->mtag.is_global) @@ -2573,9 +2577,6 @@ struct tree_struct_field_tag GTY(()) /* Size of the field. */ unsigned HOST_WIDE_INT size; - /* True if this SFT is for a field in a nested structure. */ - unsigned int in_nested_struct : 1; - /* Alias set for a DECL_NONADDRESSABLE_P field. Otherwise -1. */ alias_set_type alias_set; }; @@ -2587,7 +2588,7 @@ struct tree_struct_field_tag GTY(()) (STRUCT_FIELD_TAG_CHECK (NODE)->sft.alias_set != -1) #define SFT_ALIAS_SET(NODE) (STRUCT_FIELD_TAG_CHECK (NODE)->sft.alias_set) #define SFT_IN_NESTED_STRUCT(NODE) \ - (STRUCT_FIELD_TAG_CHECK (NODE)->sft.in_nested_struct) + (STRUCT_FIELD_TAG_CHECK (NODE)->sft.common.in_nested_struct) /* Memory Partition Tags (MPTs) group memory symbols under one common name for the purposes of placing memory PHI nodes. */