Comment 67 for bug 908508

Revision history for this message
In , Landry-openbsd (landry-openbsd) wrote :

(In reply to Landry Breuil from comment #55)
> Created attachment 600506
> Use YARR interpreter instead of PCRE on platforms where YARR JIT is not
> supported (m-c patch)
>
> That was it, with this new patch the build goes past js/. Changes from last
> patch :
> - update after landing of #700822, ie enclose use of execAlloc_ withing #if
> ENABLE_ASSEMBLER in jscntxt.cpp/setJitHardening()
> - remove #if ENABLE_YARR_JIT in checkSyntax
> - don't readd Assertions.cpp to CPPSRCS in !ENABLE_YARR_JIT case
> - include YarrSyntaxChecker.h inconditionally in js/src/vm/RegExpObject.h
>
> Results of the build in ~10h at
> http://buildbot.rhaalovely.net/builders/mozilla-central-macppc/builds/61

That particular build failed for other reasons (repository in a mixed state) but i've built successfully with that patch, and m-c works fine on OpenBSD/ppc. Chris, can you review it ? i see no reason to wait for 661974, and it would fix the build for exotic architectures.