Closed
Bug 669564
Opened 14 years ago
Closed 14 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•14 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•14 years ago
|
||
| Assignee | ||
Updated•14 years ago
|
Attachment #544165 -
Flags: review?(bhackett1024)
| Assignee | ||
Comment 2•14 years ago
|
||
After TI merge, math-jit-tests always become failure on Win64...
Comment 3•14 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•14 years ago
|
||
Whiteboard: [fixed-in-tracemonkey]
| Assignee | ||
Comment 5•14 years ago
|
||
Backed out by http://hg.mozilla.org/tracemonkey/rev/9863e9639d80
Whiteboard: [fixed-in-tracemonkey]
| Assignee | ||
Comment 6•14 years ago
|
||
It seems that XMM (FP) registers need continuity order... I am investigating again.
| Assignee | ||
Comment 7•14 years ago
|
||
Attachment #544165 -
Attachment is obsolete: true
| Assignee | ||
Updated•14 years ago
|
Attachment #547373 -
Flags: review?(dvander)
Updated•14 years ago
|
Attachment #547373 -
Flags: review?(dvander) → review+
| Assignee | ||
Comment 8•14 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•14 years ago
|
||
Status: NEW → RESOLVED
Closed: 14 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
•