Closed Bug 1644472 Opened 4 years ago Closed 4 years ago

Optimize some RegExp intrinsics in CacheIR and Warp

Categories

(Core :: JavaScript Engine: JIT, task, P2)

task

Tracking

()

RESOLVED FIXED
mozilla79
Tracking Status
firefox79 --- fixed

People

(Reporter: jandem, Assigned: jandem)

References

(Blocks 1 open bug)

Details

Attachments

(4 files)

I have some patches for IsRegExpObject and RegExp{Matcher,Searcher,Tester} that improve Warp perf on octane-regexp.

I tried to keep this generic in case we add a similar function for another Class
in the future.

The CacheIR code is just a VM call that's slightly faster than the CallNative path.
The main benefit is that this lets Warp use the optimized regexp stubs.

At some point it would be nice to call the same code stubs from CacheIR.

Depends on D79040

These just do a Shape guard against a Shape stored in the Realm. If the guard
fails we do a callWithABI to a function that updates the shape.

Depends on D79263

Keywords: leave-open
Pushed by jdemooij@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/a5845925ecf9
part 1 - Optimize IsRegExpObject intrinsic in CacheIR and Warp. r=evilpie
https://hg.mozilla.org/integration/autoland/rev/f5d92b5d503e
part 2 - Optimize RegExpMatcher, RegExpSearcher, RegExpTester intrinsics in CacheIR and Warp. r=iain
Pushed by jdemooij@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/57fb53405c05
part 3 - Optimize SubstringKernel intrinsic in CacheIR and Warp. r=evilpie
https://hg.mozilla.org/integration/autoland/rev/798a5cd87e54
part 4 - Optimize RegExpPrototypeOptimizable and RegExpInstanceOptimizable intrinsics. r=iain
Keywords: leave-open
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla79
Regressions: 1645968
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: