Closed Bug 722917 Opened 10 years ago Closed 9 years ago

IonMonkey: TI is not enabled for ARM

Categories

(Core :: JavaScript Engine, defect)

ARM
Linux
defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: mjrosenb, Unassigned)

Details

Attachments

(1 file, 1 obsolete file)

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)
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)
Attachment #593266 - Flags: review?(dvander)
Attachment #593266 - Flags: review?(dvander) → review+
http://hg.mozilla.org/projects/ionmonkey/rev/d283b00c1f73
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.