This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [Patch] Emit 64-bit bounds on 32-bit hosts with DWARF

From: Eric Botcazou <ebotcazou at adacore dot com>
To: Jakub Jelinek <jakub at redhat dot com>
Cc: gcc-patches at gcc dot gnu dot org, Ian Lance Taylor <iant at google dot com>
Date: Wed, 30 Sep 2009 13:29:18 +0200
Subject: Re: [Patch] Emit 64-bit bounds on 32-bit hosts with DWARF
References: <200907111348.11039.ebotcazou@adacore.com> <200909291408.57833.ebotcazou@adacore.com> <20090929142138.GL14664@tyan-ft48-01.lab.bos.redhat.com>

> Here is an only very lightly tested patch which does that.  So, if your
> changes are reverted (with adding a CONST_DOUBLE instead of double_int),
> then dw_val_node is suddenly only sizeof (int) + sizeof (HOST_WIDE_INT)
> on 32-bit hosts and 16 bytes on 64-bit hosts.
> Given that e.g. every dw_loc_descr_node has 2 of them and dw_attr_node one,
> I think it should save quite a lot of memory on larger CUs.

Fine, but I don't see why my changes need to be reverted, they fix a bug.

If you really want to reduce memory consumption this time, then make sure to 
measure the size of the union and install your patch on top of mine.

-- 
Eric Botcazou

References:
- Re: [Patch] Emit 64-bit bounds on 32-bit hosts with DWARF
  - From: Eric Botcazou
- Re: [Patch] Emit 64-bit bounds on 32-bit hosts with DWARF
  - From: Jakub Jelinek

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]