Re: yy_state_t

[Akim Demaille]

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!