Created attachment 583827 [details] [diff] [review]
Big endian fix for YarrJIT ignoreCase
Although PowerPC JM+TI is currently stillborne -_-, we are making use of our macroassembler for YARR and Ben Stuhl, one of our TenFourFox contributors, discovered an endianness issue with YARR JIT. It's a small patch and I figured we should push it upstream for SPARC and (future) MIPS consumers. Ben's notes:
I built js/ from 10.4Fx 9 and ran 'make check jstests'. It revealed a few failures, including several; in ecma_3/RegExp/perlstress-001.js, which can be reduced to
'A]'.match(/a]/i) == null /* should be 'A]' */
. I get the same result running the test by hand in the Web Console in 10.4Fx 9.
It turns out to be a minor endian bug in YARR; here's a patch.
Did you file a bug with WebKit too?
Not yet, I'll do that too. Thanks for the review!