Hi Paul, I'd like your input on this. Cheers!
> Le 15 oct. 2019 à 08:45, Akim Demaille <[email protected]> a écrit : > >> Le 13 oct. 2019 à 09:54, Akim Demaille <[email protected]> a écrit : >> >> Here's what it gives. Cheers! >> >> commit 3c9656ec6ea39c4f1bad4ab2c153267f61015438 >> Author: Akim Demaille <[email protected]> >> Date: Sat Oct 12 11:00:05 2019 +0200 >> >> yacc: rename types for states >> >> * data/skeletons/yacc.c (yy_state_num): Rename as... >> (yy_state_t): this. >> (yy_state_fast_t): New. >> Use it. > > Installed. > > 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.
