Closed Bug 1396767 Opened 7 years ago Closed 7 years ago

MIPS: Fix MOVF in simulator

Categories

(Core :: JavaScript Engine: JIT, defect)

Other
All
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla57
Tracking Status
firefox57 --- fixed

People

(Reporter: miran.karic, Assigned: miran.karic)

Details

Attachments

(1 file)

User Agent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/59.0.3071.104 Safari/537.36

Steps to reproduce:

When reading cc field for MOVF instruction, incorrect bits were accessed. This caused the test asm.js/testMathLib.js to fail. The uploaded patch fixes the problem with MOVF.
Attachment #8904486 - Flags: review?(lhansen)
Attachment #8904486 - Flags: checkin?
Comment on attachment 8904486 [details] [diff] [review]
0001-Bug-1396767-MIPS-Fix-MOVF-in-simulator.patch

Review of attachment 8904486 [details] [diff] [review]:
-----------------------------------------------------------------

Looks reasonable to me, though I haven't looked in detail (not enough of a MIPS expert :)
Attachment #8904486 - Flags: review?(lhansen)
Attachment #8904486 - Flags: review+
Attachment #8904486 - Flags: checkin?
Keywords: checkin-needed
Status: UNCONFIRMED → NEW
Ever confirmed: true
OS: Unspecified → All
Hardware: Unspecified → Other
(In reply to Lars T Hansen [:lth] from comment #2)
> Comment on attachment 8904486 [details] [diff] [review]
> 0001-Bug-1396767-MIPS-Fix-MOVF-in-simulator.patch
> 
> Review of attachment 8904486 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> Looks reasonable to me, though I haven't looked in detail (not enough of a
> MIPS expert :)

Thank you! Could you also assign this to me, not sure but maybe this is needed to land?
I *think* the checkin-needed should be enough but it won't hurt if you own the bug.
Assignee: nobody → miran.karic
https://hg.mozilla.org/mozilla-central/rev/8bc120c1cd33
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla57
You need to log in before you can comment on or make changes to this bug.