Closed Bug 853710 Opened 11 years ago Closed 11 years ago

IonMonkey: (ARM) optimise use of stm/ldm

Categories

(Core :: JavaScript Engine, defect)

ARM
Android
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla22

People

(Reporter: nrc, Assigned: nrc)

Details

Attachments

(1 file)

There is an unnecessary add/sub due to using |reserveStack| rather than |adjustFrame|
Attached patch patchSplinter Review
Attachment #728016 - Flags: review?(mrosenberg)
It looks like this was pushed with r=mjrosenb, but the patch was not actually reviewed:

https://hg.mozilla.org/integration/mozilla-inbound/rev/eb1e425dbf0f
Oh, that's actually a different bug. Nevermind!
(In reply to Sean Stangl [:sstangl] from comment #2)
> It looks like this was pushed with r=mjrosenb, but the patch was not
> actually reviewed:
> 
> https://hg.mozilla.org/integration/mozilla-inbound/rev/eb1e425dbf0f

Nope, the similarly-named bug 849489.
Attachment #728016 - Flags: review?(mrosenberg) → review+
https://hg.mozilla.org/mozilla-central/rev/9c98428a284e
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla22
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: