[MIPS] Some BigInt related tests failed on mips platform
Categories
(Core :: JavaScript Engine: JIT, defect, P5)
Tracking
()
| Tracking | Status | |
|---|---|---|
| firefox-esr78 | --- | unaffected |
| firefox84 | --- | unaffected |
| firefox85 | --- | unaffected |
| firefox86 | --- | wontfix |
| firefox87 | --- | wontfix |
| firefox88 | --- | fixed |
People
(Reporter: zhaojiazhong-hf, Assigned: zhaojiazhong-hf)
References
(Blocks 1 open bug, Regression)
Details
(Keywords: regression)
Attachments
(2 files, 1 obsolete file)
|
Assignee | |
Comment 1•4 years ago
|
||
On mips64 platform, some BitInt related jit-tests failed, like:
- bigint/bigint-add.js
- bigint/bigint-dec.js
- bigint/bigint-inc.js
- bigint/bigint-mul.js
- etc
And it's because some functions are not implemented on mips platform, which is introduced in bug 1679750. I will try to port the changes to mips64 platform.
|
||
Updated•4 years ago
|
|
||
Updated•4 years ago
|
|
|
||
Comment 2•4 years ago
|
||
Set release status flags based on info from the regressing bug 1679750
|
||
Updated•4 years ago
|
|
|
Assignee | |
Comment 3•4 years ago
|
||
Port D98163, D98165, D98167 to mips platform.
|
||
Updated•4 years ago
|
|
||
Updated•4 years ago
|
Add convertDoubleToPtr, move32SignExtendToPtr, spectreBoundsCheckPtr and spectreMaskIndexPtr to the MacroAssembler;
Implement MacroAssembler::branch{Add,Sub,Mul}Ptr;
|
|
||
Updated•4 years ago
|
|
||
Comment 7•4 years ago
|
||
| bugherder | ||
|
||
Updated•4 years ago
|
|
|
||
Updated•4 years ago
|
|
|
Assignee | |
Comment 8•4 years ago
|
||
It seems that this patch introduced an assert error due to misusing the scratch register. Currently, the scratch register usage on mips port is a mess, I will firstly fix this assert failure to make sure at least the tests pass.
|
|
Assignee | |
Comment 9•4 years ago
|
||
The rt operand may be the ScratchRegister and will cause an assert fail,
now change the implementation of the function to remove the assert.
|
||
Comment 10•4 years ago
|
||
|
||
Comment 11•4 years ago
|
||
| bugherder | ||
Description
•