Hi Paul, > Le 24 oct. 2019 à 02:34, Paul Eggert <[email protected]> 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!
