Closed Bug 1630090 Opened 4 years ago Closed 4 years ago

Irregexp: Call regexps from Ion

Categories

(Core :: JavaScript Engine, task, P1)

task

Tracking

()

RESOLVED FIXED
mozilla77
Tracking Status
firefox77 --- fixed

People

(Reporter: iain, Assigned: iain)

References

Details

Attachments

(2 files)

Bug 1628835 and bug 1629670 handle interpreting and compiling regexps. The last major piece of the puzzle is calling them from Ion (and Warp).

PrepareAndExecuteRegexp has to be updated to call the new engine. In preparation, this patch factors out a couple of helper functions that a) can be shared between the two versions, and b) make sense as standalone functions in the future when we remove the old implementation.

The code in this patch lines up fairly closely with the old version of PrepareAndExecuteRegexp, but not so closely that I wanted to unify them, particularly since the old version will be going away. Aside from the changes to the layout of the input data, the main difference is that the return value is now being passed in a register instead of on the stack.

Depends on D71080

Pushed by iireland@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/7efbecf663a8
Refactor PrepareAndExecuteRegexp r=jandem
https://hg.mozilla.org/integration/autoland/rev/fe4b1b625e77
Implement PrepareAndExecuteRegexp for new engine r=jandem
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla77
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: