[JSC] Yarr should use Bitmap instead of ByteVector for BoyerMoore search
Created attachment 434690 [details] Patch
Created attachment 434691 [details] Patch
Created attachment 434692 [details] Patch
Created attachment 434703 [details] Patch
Comment on attachment 434703 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=434703&action=review > Source/JavaScriptCore/yarr/YarrJIT.cpp:2428 > + auto matched = branchTestBit64(NonZero, regT2, regT0); // We can ignore upper bits since module-64 is performed. module->modulo The instruction itself performs modulo 64?
r=me
Comment on attachment 434703 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=434703&action=review Thanks! >> Source/JavaScriptCore/yarr/YarrJIT.cpp:2428 >> + auto matched = branchTestBit64(NonZero, regT2, regT0); // We can ignore upper bits since module-64 is performed. > > module->modulo > > The instruction itself performs modulo 64? Yes. https://www.felixcloutier.com/x86/bt
Committed r280544 (240171@main): <https://commits.webkit.org/240171@main>
<rdar://problem/81419055>