Closed
Bug 722917
Opened 12 years ago
Closed 12 years ago
IonMonkey: TI is not enabled for ARM
Categories
(Core :: JavaScript Engine, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: mjrosenb, Unassigned)
Details
Attachments
(1 file, 1 obsolete file)
3.60 KB,
patch
|
dvander
:
review+
|
Details | Diff | Splinter Review |
Looks like there was a typo in the old MacroAssembler that caused TI to never get turned on in my tests (or almost any other ARM tests).
Attachment #593255 -
Flags: review?(dvander)
Reporter | ||
Comment 1•12 years ago
|
||
dvander pointed out that (this == NULL) isn't valid C++, so I've pulled the check out to the calling functions.
Attachment #593255 -
Attachment is obsolete: true
Attachment #593255 -
Flags: review?(dvander)
Reporter | ||
Updated•12 years ago
|
Attachment #593266 -
Flags: review?(dvander)
Updated•12 years ago
|
Attachment #593266 -
Flags: review?(dvander) → review+
http://hg.mozilla.org/projects/ionmonkey/rev/d283b00c1f73
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•