This is the mail archive of the gcc@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: ggc_alloc_rtvec_sized allocates spaces more than necessary?

From: Laurynas Biveinis <laurynas dot biveinis at gmail dot com>
To: Richard Guenther <richard dot guenther at gmail dot com>
Cc: çä <netcasper at gmail dot com>, gcc at gcc dot gnu dot org
Date: Tue, 16 Aug 2011 08:51:38 +0300
Subject: Re: ggc_alloc_rtvec_sized allocates spaces more than necessary?
References: <CAO+NnCbQHLxWucv4hUrHUBNjg1u=9iJte_c_gy6-wniz-iHA3Q@mail.gmail.com> <CAFiYyc0JQtr9X19s929XidCVdQS3=NJRZDo4_Dk0DakHHseOqw@mail.gmail.com>

> On Mon, Aug 15, 2011 at 2:16 PM, çä <netcasper@gmail.com> wrote:
>> The size it allocates is
>>
>> Â(sizeof (struct rtvec_def) + ((NELT) - 1)) * sizeof (rtx)

>> Originally, the allocated size is
>>
>> Âsizeof (struct rtvec_def) + ((NELT) - 1) * sizeof (rtx)

Yes, this is correct, good catch.

>> Â(sizeof (struct rtvec_def) + sizeof (rtx) - 1) / sizeof (rtx) +
>> ((NELT) - 1) Â // (3)

Due to the way those macros expand, right now replacing the first arg
with "1" and the second one with straightforward "sizeof (struct
rtvec_def) + ((NELT) - 1) * sizeof (rtx)" will work and will be easier
to read than division. Liang, would you submit such patch?

Thanks again,
-- 
Laurynas

Follow-Ups:
- Re: ggc_alloc_rtvec_sized allocates spaces more than necessary?
  - From: Liang Wang

References:
- ggc_alloc_rtvec_sized allocates spaces more than necessary?
  - From: çä
- Re: ggc_alloc_rtvec_sized allocates spaces more than necessary?
  - From: Richard Guenther

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