Closed Bug 821273 Opened 12 years ago Closed 12 years ago

Regression on dart page, wrong numbers are calculated

Categories

(Core :: JavaScript Engine, defect)

Product:
Core
Component:
JavaScript Engine
Type:
defect
Priority:
Not set
Severity:
major

Tracking

()

Status:
RESOLVED DUPLICATE of bug 821788

People

(Reporter: evilpie, Assigned: bhackett1024)

References

(Depends on 1 open bug,
URL
)

Details

(Keywords: regression) 

+++ This bug was initially created as a clone of Bug #820528 +++

http://www.dartlang.org/performance/ is not only bad performance wise. We also have a quite recent regression. The graph data is not displayed properly.

Responsible push:
https://tbpl.mozilla.org/?rev=e83393045b95
Keywords: regression
Summary: Performance graph written in Dart compiled to JS is laggy → Regression on dart page, wrong numbers are calculated
Disabling ion fixes this. This could be caused by Bug 815161 or Bug 814997.
Not related to Bug 815161. Backing out that patch, doesn't solve it. Therefore I think it is bug 814997: IonMonkey: safepoints shouldn't artificially extend vreg lifetimes. That's the reason I assigned you Brain.
Assignee: general → bhackett1024
With an x64 debug browser the page is broken for me and is fixed by the patch in bug 821788.  While the problem there is with eliminated phis, both phi and resume point operand elimination use the folded bits on MIR instructions, which were not being set correctly for inlined calls.
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.