Closed Bug 564414 Opened 14 years ago Closed 14 years ago

XP_MACOSX is not defined on Mac

Categories

(Core :: JavaScript Engine, defect)

Other Branch
defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: igor, Assigned: igor)

References

Details

(Whiteboard: fixed-in-tracemonkey)

Attachments

(1 file, 1 obsolete file)

Currently the JS build system does not define XP_MACOSX when building on MACOSX meaning that few mac-specific optimizations are not picked up.
Attached patch v1 (obsolete) — Splinter Review
This is what I have submitted to the try server.
Attached patch v2Splinter Review
Try server revealed that some code got rotten and the v2 fixes compilation errors and warnings comming from defining XP_MACOSX
Attachment #444073 - Attachment is obsolete: true
Attachment #444245 - Flags: review?(ted.mielczarek)
Blocks: 237006
Ted: This patch blocks the bug 516832, beta1 blocker. So could you review the patch quickly?
Attachment #444245 - Flags: review?(ted.mielczarek) → review+
http://hg.mozilla.org/tracemonkey/rev/331f0bb3bbd6
Whiteboard: fixed-in-tracemonkey
http://hg.mozilla.org/mozilla-central/rev/331f0bb3bbd6
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: