[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: struct optimizations

From: Ludovic Courtès
Subject: Re: struct optimizations
Date: Tue, 26 Jan 2010 00:13:36 +0100
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/23.1 (gnu/linux)


Andy Wingo <address@hidden> writes:

> Some comments:
>     -#define SCM_VTABLE_FLAG_RESERVED_0 (1L << 5)
>     -#define SCM_VTABLE_FLAG_RESERVED_1 (1L << 6)
>     +#define SCM_VTABLE_FLAG_SIMPLE (1L << 5) /* instances of this vtable 
> have only "pr" fields */
>     +#define SCM_VTABLE_FLAG_SIMPLE_RW (1L << 6) /* instances of this vtable 
> have only "pw" fields */
> The comments do not appear to be correct, given a later check:
>     +                   && SCM_VTABLE_FLAG_IS_SET (vtable, 
>     +                   && p < SCM_STRUCT_DATA_REF (vtable, 
> scm_vtable_index_size)))
>     +    /* The fast path: HANDLE is a struct with only "p" fields.  */
>     +    data[p] = SCM_UNPACK (val);
> It seems that currently SIMPLE is for all pr *or* all pw slots. But we
> should be more orthogonal than that; let's have SIMPLE be for having all
> slots be readable p slots. (Note that this still allows a mix of
> readable and writable slots.)


> Then perhaps we can change SIMPLE_RW to be MUTABLE or something, to
> indicate that all slots of this object are mutable.

I ended up keeping the name ‘SIMPLE_RW’ because in theory there could be
non-‘p’ mutable fields, i.e., non-simple structs with mutable fields,
which aren’t interesting.


reply via email to

[Prev in Thread] Current Thread [Next in Thread]