Comment 53 for bug 908508

Revision history for this message
In , Christopher-leary (christopher-leary) wrote :

(In reply to Landry Breuil from comment #41)
> Apart checkSyntax, it should be okay. Chris, any idea what
> to do about that, and would it be so wrong to return true, given that it
> worksforme ?

Hey Landry, sorry for the delay on this. We have to do something about checkSyntax. I need to create a patch that disables YARR-JIT for testing anyway, so I'll try to get you a patch that rips out PCRE (leaving it in obviously hasn't been working out) and adds a preprocessor option to disable YARR-JIT entirely and rely so PPC can rely on YARR-interp. I started doing this fallback as a runtime option in bug 661974, but adding a preprocessor seam as well shouldn't be tough.