Closed
Bug 1165392
Opened 9 years ago
Closed 9 years ago
Fix some unboxed array bugs
Categories
(Core :: JavaScript Engine, defect)
Core
JavaScript Engine
Tracking
()
RESOLVED
FIXED
mozilla41
Tracking | Status | |
---|---|---|
firefox41 | --- | fixed |
People
(Reporter: bhackett1024, Assigned: bhackett1024)
References
Details
Attachments
(5 files)
22.89 KB,
patch
|
jandem
:
review+
|
Details | Diff | Splinter Review |
9.53 KB,
patch
|
jandem
:
review+
|
Details | Diff | Splinter Review |
3.48 KB,
patch
|
jandem
:
review+
|
Details | Diff | Splinter Review |
20.25 KB,
patch
|
jandem
:
review+
|
Details | Diff | Splinter Review |
21.44 KB,
patch
|
jandem
:
review+
|
Details | Diff | Splinter Review |
The attached patch fixes some bugs with unboxed arrays I found while testing.
Attachment #8606387 -
Flags: review?(jdemooij)
Assignee | ||
Comment 1•9 years ago
|
||
The baseline SETELEM caches for unboxed arrays were never being used, and are pretty broken.
Assignee: nobody → bhackett1024
Attachment #8607239 -
Flags: review?(jdemooij)
Assignee | ||
Comment 2•9 years ago
|
||
Some more code that is responsible for handling effectless lookups needs to be aware of unboxed arrays.
Attachment #8607240 -
Flags: review?(jdemooij)
Assignee | ||
Comment 3•9 years ago
|
||
Add Ion caches for GETELEM and length GETPROP on unboxed arrays.
Attachment #8607241 -
Flags: review?(jdemooij)
Assignee | ||
Comment 4•9 years ago
|
||
Handle unboxed arrays in the Ion optimized version of array.concat
Attachment #8607242 -
Flags: review?(jdemooij)
Comment 5•9 years ago
|
||
Comment on attachment 8606387 [details] [diff] [review] patch Review of attachment 8606387 [details] [diff] [review]: ----------------------------------------------------------------- ::: js/src/jit/MIR.h @@ +7785,1 @@ > bool congruentTo(const MDefinition* ins) const override { congruentTo should check checkNative_ right?
Attachment #8606387 -
Flags: review?(jdemooij) → review+
Updated•9 years ago
|
Attachment #8607239 -
Flags: review?(jdemooij) → review+
Comment 6•9 years ago
|
||
Comment on attachment 8607240 [details] [diff] [review] improve JIT lookups Review of attachment 8607240 [details] [diff] [review]: ----------------------------------------------------------------- ::: js/src/jsobj.cpp @@ +2812,5 @@ > } > + } else if (obj->is<UnboxedArrayObject>()) { > + if (obj->as<UnboxedArrayObject>().containsProperty(cx, id)) { > + *objp = obj; > + MarkNonNativePropertyFound<NoGC>(propp); Btw, is fixing the 0x1 shape thing still on your radar? :)
Attachment #8607240 -
Flags: review?(jdemooij) → review+
Updated•9 years ago
|
Attachment #8607241 -
Flags: review?(jdemooij) → review+
Updated•9 years ago
|
Attachment #8607242 -
Flags: review?(jdemooij) → review+
https://hg.mozilla.org/mozilla-central/rev/57dce88fc620
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla41
You need to log in
before you can comment on or make changes to this bug.
Description
•