[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: yy_state_t
From: |
Akim Demaille |
Subject: |
Re: yy_state_t |
Date: |
Thu, 24 Oct 2019 18:18:16 +0200 |
Hi Paul,
> Le 24 oct. 2019 à 02:34, Paul Eggert <address@hidden> a écrit :
>
> On 10/21/19 2:27 AM, Akim Demaille wrote:
>> I'm ambivalent for glr.c and lalr1.cc, since they already use int everywhere
>> for states, and are unlikely to benefit from the width reduction. Except
>> for the auxiliary state stack for LAC in lalr1.cc.
>> I do like the fact that the skeletons are more consistent between them (I
>> will rename the types for states afterwards), but still... I'd appreciate
>> other opinions.
>
> If they're unlikely to benefit from width reduction, I wouldn't bother. (I
> don't have a strong feeling about it either.)
Thanks! So I installed only the patch for lalr1.cc, given that it reduces the
size of the LAC stack. I could have used a conditional to change _only_ when
LAC is enabled, but that's too much effort IMHO.
Cheers!
- yy_state_t, Akim Demaille, 2019/10/12
- Re: yy_state_t, Paul Eggert, 2019/10/12
- Re: yy_state_t, Akim Demaille, 2019/10/13
- Re: yy_state_t, Akim Demaille, 2019/10/15
- Re: yy_state_t, Akim Demaille, 2019/10/21
- Re: yy_state_t, Paul Eggert, 2019/10/23
- Re: yy_state_t,
Akim Demaille <=
- Re: yy_state_t, Akim Demaille, 2019/10/25