1642493 - Irregexp: Tidy up loose ends

Status: RESOLVED FIXED

(Core :: JavaScript Engine, task, P1)

Description

[Iain Ireland [:iain]]

Now that the new engine has landed and made it into 78, we don't need the old engine any more, and we can clean up a variety of small things.

Comment 1

[Iain Ireland [:iain]]

Attachment: Bug 1642493: Add case-folding testcase r=mgaudet

I wrote this testcase while fixing an issue with upstream V8's handling of non-unicode ignoreCase RegExps. (See here: https://github.com/v8/v8/commit/3fab9d05cf34a7f0bc0e9405729ab8b78c0671ac)

I'm finally getting around to landing it.

(There will be another patch in the future to fix an equivalent issue with back-references, but it's waiting on upstream review.)

Comment 2

[Iain Ireland [:iain]]

Attachment: Bug 1642493: Add named capture proxy testcase r=mgaudet

I wrote this testcase while trying to figure out the spec for named captures, and discovered that V8 was not handling it properly. Our implementation does the right thing here, so I'm landing this testcase to make sure it stays that way.

Depends on D77725

Comment 3

[Iain Ireland [:iain]]

Attachment: Bug 1642493: Remove old copy of irregexp and ENABLE_NEW_REGEXP ifdefs r=mgaudet

This code no longer sparks joy.

Depends on D77726

Comment 4

[Iain Ireland [:iain]]

Attachment: Bug 1642493: Clean up remaining references to ENABLE_NEW_REGEXP and old irregexp r=mgaudet

I kept a running list of code that would be dead once we removed the old engine. This is what was on that list.

Note: The regexp_parse tests used a testing function that parsed a regexp and returned an AST to compare against the expected AST. They are being deleted because the tests are fragile in the face of upstream changes, not particularly useful, and a lot of work to convert.

Depends on D77727

Comment 5

[Iain Ireland [:iain]]

Attachment: Bug 1642493: Rename new-regexp to irregexp r=mgaudet

It's not new- any more.

This patch moves the files, replaces new-regexp with irregexp throughout the code, re-alphabetizes #includes, and adds a few missing #includes to RegExpTypes.h and util/vector.h that were exposed by the reordering.

Depends on D77728

Comment 6

[Iain Ireland [:iain]]

Attachment: Bug 1642493: Move RegExpStackScope inside irregexp r=mgaudet

This is the only place where any code outside the irregexp directory directly includes a V8 header. Tidying it up makes the next patch easier.

Depends on D77729

Comment 7

[Iain Ireland [:iain]]

Attachment: Bug 1642493: Separate imported files from shim r=mgaudet

I've been thinking about doing this for a while, and finally got around to doing it.

This patch moves the files that are directly imported from V8 into their own subdirectory of irregexp, to make it more obvious which code is part of V8 and which code is part of the shim. I also took the opportunity to rename the shim files to follow our naming conventions (FileName.cpp) instead of V8's (file-name.cc).

Most of this code is mechanical. The mildly interesting parts are:

1. In import-irregexp.py, when figuring out where the regexp shim include should go, we no longer have to work quite as hard to alphabetize it, because irregexp/RegExpShim.h will always come after irregexp/imported/*. Instead, we just keep scanning until we find a newline.

2. Some of the V8 code #includes "regexp-macro-assembler-arch.h". Up until now, that's the name we used for the header file for our RegExpMacroAssembler implementation. Now that the implementation lives in a separate directory from the V8 code, I've renamed the header to RegExpNativeMacroAssembler.h (to match the .cpp file), and added a stub regexp-macro-assembler-arch.h header in imported that #includes the real thing.

In the next patch, we take advantage of the new directory structure to get more precise with clang-format-ignore.

Depends on D77730

Comment 8

[Iain Ireland [:iain]]

Attachment: Bug 1642493: Apply clang-format to shim files r=mgaudet

Now this code is less of a weird outlier.

Depends on D77731

Comment 9

[Iain Ireland [:iain]]

Attachment: Bug 1642493: Remove obsolete directory from license.html r=mhoye

The code that was previously in new-regexp has been moved into irregexp, and the code in irregexp has been deleted. Update the license file accordingly.

Depends on D77732

Comment 10

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/792796a61455
Add case-folding testcase r=mgaudet
https://hg.mozilla.org/integration/autoland/rev/3e8cda5dd726
Add named capture proxy testcase r=mgaudet
https://hg.mozilla.org/integration/autoland/rev/4436d60dd86a
Remove old copy of irregexp and ENABLE_NEW_REGEXP ifdefs r=mgaudet
https://hg.mozilla.org/integration/autoland/rev/5bdd0c3c33ef
Clean up remaining references to ENABLE_NEW_REGEXP and old irregexp r=mgaudet
https://hg.mozilla.org/integration/autoland/rev/737f8abf8315
Rename new-regexp to irregexp r=mgaudet
https://hg.mozilla.org/integration/autoland/rev/3626007f27aa
Move RegExpStackScope inside irregexp r=mgaudet
https://hg.mozilla.org/integration/autoland/rev/7aa7ba13b1bd
Separate imported files from shim r=mgaudet
https://hg.mozilla.org/integration/autoland/rev/8134bf0eafd7
Apply clang-format to shim files r=mgaudet

Comment 11

[Natalia Csoregi [:nataliaCs]]

https://hg.mozilla.org/mozilla-central/rev/792796a61455
https://hg.mozilla.org/mozilla-central/rev/3e8cda5dd726
https://hg.mozilla.org/mozilla-central/rev/4436d60dd86a
https://hg.mozilla.org/mozilla-central/rev/5bdd0c3c33ef
https://hg.mozilla.org/mozilla-central/rev/737f8abf8315
https://hg.mozilla.org/mozilla-central/rev/3626007f27aa
https://hg.mozilla.org/mozilla-central/rev/7aa7ba13b1bd
https://hg.mozilla.org/mozilla-central/rev/8134bf0eafd7

Comment 12

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/87944df44cda
Remove obsolete directory from license.html r=mhoye

Comment 13

[Bogdan Tara[:bogdan_tara | bogdant]]

https://hg.mozilla.org/mozilla-central/rev/87944df44cda