Re: [PATCH] Unpack bools in ACLUserData

On 24/01/2013 5:22 a.m., Kinkie wrote:
> Hi,
> this patch converts the unsinged integers in ACLUserData to unpacked bools.
>
> --
> /kinkie

Are we doing this to reduce the memory footprint? or only to simplify
the code?
This step goes some way towards memory savings by halving the memory
footprint. To reduce the memory footprint by another half again we
should be using bool:1.

In micro-benchmarks I ran 100,000,000 X.x=1 assignments; uint->bool
saves ~20 to 100 ms, bool->bool:1 saves ~5 to -90 ms more)

Amos
Received on Thu Jan 24 2013 - 01:58:25 MST