Open Bug 1440621 Opened 6 years ago Updated 2 years ago

[Ion cleanup] Remove splitTagForTest()

Categories

(Core :: JavaScript Engine: JIT, enhancement, P3)

Product:
Core
Component:
JavaScript Engine: JIT
Type:
enhancement

Tracking

()

People

(Reporter: lth, Unassigned)

References

Details

The API masm.splitTagForTest() is awkward to use because it requires an output register but no output register is generally available, so we end up overloading the scratch registers.  This has been the source of bugs (bug 1438800) and can be the source of bugs in the future.

In bug 1438800 jandem comments: "I think eventually we should remove splitTagForTest completely and just pass an explicit scratch register", the implication being that all uses are converted to use masm.extractTag() instead.
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.