Closed Bug 550811 Opened 11 years ago Closed 11 years ago

bustage on Windows x64 build for js-ctypes

Categories

(Core :: js-ctypes, defect)

x86_64
Windows Vista
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla1.9.3a3

People

(Reporter: m_kato, Assigned: m_kato)

References

Details

Attachments

(1 file)

After landing bug 538216, there is three problems.

- since uname in mozilla-build is 32-bit process, uname cannot return correct host.  So we should add --build and --host for configure of libffi
- msvcc.sh needs -m64 for x64 compiler.
- disable some warnings (different size cast etc) due to -WX argument.
Attached patch patch v1Splinter Review
Attachment #431031 - Flags: review?(ted.mielczarek)
Attachment #431031 - Flags: review?(ted.mielczarek) → review+
The msvcc.sh change will need to wind up upstream.
landed after merging benjamin's check in.
http://hg.mozilla.org/mozilla-central/rev/c82a85fe6cc3
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Summary: bustage on Windows x64 buiild for js-ctypes → bustage on Windows x64 build for js-ctypes
So, hang on. Question. The original author of msvcc.sh had a case for -m64, which translated the flag into calling cl64.exe and link64.exe (or something like that; maybe it was cc and link from the separate VC directory where the 64bit tools are installed).

Is that not required now? Do cl and link know that -m64 means "call the 64bit versions"? Or did MS just roll cl64 and link64 into cl and link directly?
When building for x64 on Windows, we require that you use a different MozillaBuild batch file, which puts the x64 compilers (they're separate binaries) in your PATH.
Ah, ok. Why is the -m64 even required, then?
(In reply to comment #6)
> Ah, ok. Why is the -m64 even required, then?

MASM for x64 is ml64.exe, not ml.exe.  msvcc.sh sets ml is ml64.exe if using -m64.
(In reply to comment #7)
> MASM for x64 is ml64.exe, not ml.exe.  msvcc.sh sets ml is ml64.exe if using
> -m64.

Oh, duh. I thought I removed that bit when I wrote the wrapper. Evidently I left it in. :)

Thanks!
Flags: in-testsuite-
Target Milestone: --- → mozilla1.9.3a3
You need to log in before you can comment on or make changes to this bug.