
7 Nov
2010
7 Nov
'10
5:35 a.m.
On Sun, Nov 7, 2010 at 4:53 AM, Bryan O'Sullivan
On Sat, Nov 6, 2010 at 3:11 PM, Antoine Latter
wrote: Assuming we move the Parsec 'many' implementation into the Alternative class definition, this should have no/little impact on performance for all the combinators mentioned by Stephen.
I can't speak for Parsec, but I certainly have a local definition of "many" in attoparsec, because the Alternative version isn't inlined. Makes a biggish difference to performance, though I do not recall the magnitude (2x?).
Ah, drat. However it looks like Parsec.Prim.many is not inlined. I thought at the instance declaration site I could include an INLINE pragma. Does that not work as one would hope? Antoine