Support Win64 (LLP64 model) for xpcom/ds

RESOLVED FIXED

Status

()

RESOLVED FIXED
10 years ago
10 years ago

People

(Reporter: m_kato, Assigned: m_kato)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

Created attachment 355914 [details] [diff] [review]
patch v1

Microsoft VC++ Compiler for Windows 64bit uses LLP64 model. 
PtrBits in xpcom/ds is not compatible for LLP64.
(Assignee)

Updated

10 years ago
Attachment #355914 - Flags: review?(benjamin)

Updated

10 years ago
Attachment #355914 - Flags: review?(benjamin) → review-

Comment 1

10 years ago
Comment on attachment 355914 [details] [diff] [review]
patch v1

I think JSUWord is what you want here... PtrBits should be an integer type the same size as void*, right?
Created attachment 361968 [details] [diff] [review]
patch v2

I think that best way is that using PRUword.

If you say JSUword is best way, jstypes have to include on some headers. (bug 478190 for /mozilla/dom tree etc)
Attachment #355914 - Attachment is obsolete: true
Attachment #361968 - Flags: review?(benjamin)

Comment 3

10 years ago
Comment on attachment 361968 [details] [diff] [review]
patch v2

Yes, exactly, PRUword for mozilla code, JSUword for spidermonkey.
Attachment #361968 - Flags: review?(benjamin) → review+
landed
http://hg.mozilla.org/mozilla-central/rev/f939168ee9f4
Status: ASSIGNED → RESOLVED
Last Resolved: 10 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.