Closed
Bug 853710
Opened 11 years ago
Closed 11 years ago
IonMonkey: (ARM) optimise use of stm/ldm
Categories
(Core :: JavaScript Engine, defect)
Tracking
()
RESOLVED
FIXED
mozilla22
People
(Reporter: nrc, Assigned: nrc)
Details
Attachments
(1 file)
2.53 KB,
patch
|
mjrosenb
:
review+
|
Details | Diff | Splinter Review |
There is an unnecessary add/sub due to using |reserveStack| rather than |adjustFrame|
Assignee | ||
Comment 1•11 years ago
|
||
Attachment #728016 -
Flags: review?(mrosenberg)
Comment 2•11 years ago
|
||
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
Comment 3•11 years ago
|
||
Oh, that's actually a different bug. Nevermind!
Assignee | ||
Comment 4•11 years ago
|
||
(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.
Assignee | ||
Comment 5•11 years ago
|
||
try push: https://tbpl.mozilla.org/?tree=Try&rev=5468d9f9695c
Updated•11 years ago
|
Attachment #728016 -
Flags: review?(mrosenberg) → review+
Assignee | ||
Comment 6•11 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/9c98428a284e
Comment 7•11 years ago
|
||
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.
Description
•