Closed Bug 1544631 Opened 7 months ago Closed 7 months ago

mips branchTestBigInt negPtr generateVMWrapper

Categories

(Core :: JavaScript Engine, defect)

68 Branch
defect
Not set

Tracking

()

RESOLVED FIXED
mozilla68
Tracking Status
firefox68 --- fixed

People

(Reporter: qiaopengcheng-hf, Assigned: qiaopengcheng-hf)

References

(Depends on 1 open bug)

Details

Attachments

(2 files, 2 obsolete files)

User Agent: Mozilla/5.0 (X11; Linux mips64; rv:52.0) Gecko/20100101 Firefox/52.0

Steps to reproduce:

compiling the JS code on mips-linux

Actual results:

compiling the JS code on mips-linux, these functions:
branchTestBigInt, negPtr, generateVMWrapper
are not properly ported.

Expected results:

should compile sucessfully

Attachment #9058471 - Flags: checkin?(jdemooij)
Attachment #9058471 - Attachment is obsolete: true
Attachment #9058471 - Flags: checkin?(jdemooij)
Attachment #9058476 - Flags: checkin?(jdemooij)

Hi, please request the review+ before landing this.

Flags: needinfo?(qiaopengcheng-hf)

(In reply to Daniel Varga [:dvarga] from comment #3)

Hi, please request the review+ before landing this.

There is no selection for reviewer after review+.

Flags: needinfo?(qiaopengcheng-hf)
Comment on attachment 9058476 [details] [diff] [review]
Fix functions branchTestBigInt negPtrgenerateVMWrapper on MIPS

I remeber that seletion review+ will ask a reviewer.
But now I can't find it.
Flags: needinfo?(dvarga)
Attachment #9058476 - Flags: checkin?(jdemooij) → review+

This still needs to be reviewed by jdemoolj.

Flags: needinfo?(dvarga) → needinfo?(jdemooij)

Please post patches using Phabricator nowadays, https://moz-conduit.readthedocs.io/en/latest/phabricator-user.html

Flags: needinfo?(jdemooij)
Attachment #9058476 - Flags: review+

qiaopengcheng, thank you for the patch. Please note that MIPS support may be dropped in the future.

(In reply to Mike Conca [:mconca] from comment #8)

qiaopengcheng, thank you for the patch. Please note that MIPS support may be dropped in the future.

Our company wants to maintains the mips and had expressed within the dev-tech-js-engine-internals@lists.mozilla.org

Fix functions: branchTestBigInt, negPtr, generateVMWrapper on MIPS.

Pushed by jdemooij@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/00719f1766c1
Fix functions: branchTestBigInt, negPtr, generateVMWrapper on MIPS. r=jandem
Status: UNCONFIRMED → RESOLVED
Closed: 7 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla68
Assignee: nobody → qiaopengcheng-hf

Bug 1544631 - part2: amend Bug-1544631 for fixing mips32.

Comment on attachment 9069168 [details]
Bug 1544631 - part2: amend Bug-1544631 for fixing mips32.

Revision D33402 was moved to bug 1556197. Setting attachment 9069168 [details] to obsolete.

Attachment #9069168 - Attachment is obsolete: true
You need to log in before you can comment on or make changes to this bug.