Closed
Bug 669564
Opened 13 years ago
Closed 13 years ago
XMM6 and XMM7 aren't scratch register on Win64 ABI
Categories
(Core :: JavaScript Engine, defect)
Tracking
()
RESOLVED
FIXED
mozilla8
People
(Reporter: m_kato, Assigned: m_kato)
References
Details
Attachments
(1 file, 1 obsolete file)
1.51 KB,
patch
|
dvander
:
review+
|
Details | Diff | Splinter Review |
On Win64 ABI, scratch register for FP is xmm0-xmm5. But Unix x86-64 ABI is xmm0-xmm7. So Win64 sometime causes jit-tests failure. So we don't include xmm6 and xmm7 as scratch registers on Win64.
Assignee | ||
Updated•13 years ago
|
Summary: XMM6 and XMM7 isn't scratch register on Win64 ABI → XMM6 and XMM7 aren't scratch register on Win64 ABI
Assignee | ||
Comment 1•13 years ago
|
||
Assignee | ||
Updated•13 years ago
|
Attachment #544165 -
Flags: review?(bhackett1024)
Assignee | ||
Comment 2•13 years ago
|
||
After TI merge, math-jit-tests always become failure on Win64...
Comment 3•13 years ago
|
||
Comment on attachment 544165 [details] [diff] [review] fix Review of attachment 544165 [details] [diff] [review]: -----------------------------------------------------------------
Attachment #544165 -
Flags: review?(bhackett1024) → review+
Assignee | ||
Comment 4•13 years ago
|
||
http://hg.mozilla.org/tracemonkey/rev/d870a3d2a5ff
Whiteboard: [fixed-in-tracemonkey]
Assignee | ||
Comment 5•13 years ago
|
||
Backed out by http://hg.mozilla.org/tracemonkey/rev/9863e9639d80
Whiteboard: [fixed-in-tracemonkey]
Assignee | ||
Comment 6•13 years ago
|
||
It seems that XMM (FP) registers need continuity order... I am investigating again.
Assignee | ||
Comment 7•13 years ago
|
||
Attachment #544165 -
Attachment is obsolete: true
Assignee | ||
Updated•13 years ago
|
Attachment #547373 -
Flags: review?(dvander)
Updated•13 years ago
|
Attachment #547373 -
Flags: review?(dvander) → review+
Assignee | ||
Comment 8•13 years ago
|
||
http://hg.mozilla.org/integration/mozilla-inbound/rev/2ebe327fac38 Also, I will handle same issue for TI repository if necessary.
Whiteboard: [inbound]
Comment 9•13 years ago
|
||
http://hg.mozilla.org/mozilla-central/rev/2ebe327fac38
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Whiteboard: [inbound]
Target Milestone: --- → mozilla8
You need to log in
before you can comment on or make changes to this bug.
Description
•