Closed
Bug 1136737
Opened 9 years ago
Closed 6 years ago
Scalar Replacement: IsArrayEscaped should support MConvertElementsToDoubles
Categories
(Core :: JavaScript Engine: JIT, defect, P5)
Core
JavaScript Engine: JIT
Tracking
()
RESOLVED
DUPLICATE
of bug 1420172
People
(Reporter: nbp, Unassigned, Mentored)
References
Details
Attachments
(1 file)
|
1003 bytes,
patch
|
jandem
:
review+
|
Details | Diff | Splinter Review |
A lot of simple test case failed to benefit from scalar replacement because we are only allocating arrays with double elements, in which case such guard is inserted in the code. This guard should be replaced by a guard on all elements of the array which is being inlined.
|
Reporter | |
Comment 1•9 years ago
|
||
Attachment #8569218 -
Flags: review?(jdemooij)
|
||
Updated•9 years ago
|
Attachment #8569218 -
Flags: review?(jdemooij) → review+
|
||
Updated•8 years ago
|
Priority: -- → P5
|
|
Reporter | |
Comment 3•8 years ago
|
||
(In reply to Tom Schuster [:evilpie] from comment #2) > Nicolas do you want to finish this? Feel free to take over, if you want.
Mentor: nicolas.b.pierron
Flags: needinfo?(nicolas.b.pierron)
|
|
||
Comment 4•6 years ago
|
||
Patch for bug 1420172 fixes this.
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → DUPLICATE
You need to log in
before you can comment on or make changes to this bug.
Description
•