Abstract much of RMATCH into MatchStack functions I'm preparing to further cleanup "recursion" in match(), starting by making RMATCH() much much smaller. Currently I'm inlining the frame management (since that's how it was before). I'm not sure that will be a win in the end. This patch actually caused my SunSpider results to stabilize, showing no slowdowns, a 9.3% improvement in regex-dna, and a 1.0% win over all (combined improvements from this and bug 15996)
Created attachment 17290 [details] abstract RMATCH into MatchStack functions
Created attachment 17291 [details] abstract RMATCH into MatchStack functions
Comment on attachment 17291 [details] abstract RMATCH into MatchStack functions The typedef should not be necessary in c++. +typedef struct matchframe { nor should this explicit struct + struct matchframe* prevframe; r=me.
Landed.