> This is a nice improvement. Even though we're in the SLP realm I would > still add an assert that documents that we're indeed operating with > pow2_p (NPATTERNS) and some comment as to why we can use AND. > Sure we're doing exact_log2 et al later anyway, just to make things > clearer.
Actually no assert necessary, just a comment like: /* As NPATTERNS is always a power of two, we can ..." */ Regards Robin