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.)

Reply via email to