Closed
Bug 587321
Opened 14 years ago
Closed 14 years ago
YARR landing broke mingw compilation
Categories
(Core :: JavaScript Engine, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: jacek, Assigned: jacek)
References
Details
Attachments
(1 file)
(deleted),
patch
|
cdleary
:
review+
|
Details | Diff | Splinter Review |
There are two problems. One is 64-bit specific. We try to build TrampolineMasmX64.asm on Windows target, but it's compiler-specific, not OS-specific. Changing |ifeq ($(OS_ARCH),WINNT)| to |ifdef _MSC_VER| fixes the problem.
The second problem is compilation of Assertions.cpp. It mixes compiler and platform checks. Includes are guarded by compiler check, while _CrtDbgReport call is guarded by OS check. It results in not including crtdbg.h and trying to use _CrtDbgReport on mingw. The best fix would be to change includes guard, but unfortunately it fails because of lack of support for debugging API on mingw. It should be fixed on their side, but I think it's fine to simply not use _CrtDbgReport on mingw and change the second guard.
Assignee | ||
Updated•14 years ago
|
Attachment #465992 -
Flags: review?(cdleary)
Assignee | ||
Updated•14 years ago
|
Assignee: general → jacek
Assignee | ||
Updated•14 years ago
|
Attachment #465992 -
Attachment is patch: true
Comment 1•14 years ago
|
||
Comment on attachment 465992 [details] [diff] [review]
fix v1.0
Looks good, though the trampoline was already removed in bug 586887. I'll go ahead and commit the win->msvc change.
Attachment #465992 -
Flags: review?(cdleary) → review+
Assignee | ||
Comment 2•14 years ago
|
||
Thanks for the review, pushed to TM:
http://hg.mozilla.org/tracemonkey/rev/311ae2addf12
Whiteboard: fixed-in-tracemonkey
Assignee | ||
Comment 3•14 years ago
|
||
Merged to m-c:
http://hg.mozilla.org/mozilla-central/rev/311ae2addf12
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Whiteboard: fixed-in-tracemonkey
You need to log in
before you can comment on or make changes to this bug.
Description
•