Closed
Bug 1579157
Opened 5 years ago
Closed 5 years ago
Cranelift: proper support of dynamic heaps
Categories
(Core :: JavaScript: WebAssembly, defect, P2)
Core
JavaScript: WebAssembly
Tracking
()
RESOLVED
FIXED
mozilla71
| Tracking | Status | |
|---|---|---|
| firefox71 | --- | fixed |
People
(Reporter: bbouvier, Assigned: bbouvier)
References
Details
Attachments
(1 file)
Cranelift tests are failing with --wasm-disable-huge-memory.
One issue I've seen is a very recent regression in Cranelift, that has not landed yet in Spidermonkey.
One other issue involves growing the heap, since there's an explicit comment in the code that the heap base is supposed to fixed: the wasm/resizing.js test is failing on inbound, probably because of this.
|
Assignee | |
Updated•5 years ago
|
Priority: -- → P2
|
|
Assignee | |
Comment 1•5 years ago
|
||
|
||
Updated•5 years ago
|
Attachment #9091036 -
Attachment description: Bug 1579157: Cranelift: mark Heap as read-only only in the huge memory case; r?rhunt → Bug 1579157: Cranelift: mark Heap as read-only only in the huge memory case; r=rhunt
Pushed by bbouvier@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/c1fc5c5e39c0 Cranelift: mark Heap as read-only only in the huge memory case; r=rhunt
|
||
Comment 3•5 years ago
|
||
| bugherder | ||
Status: NEW → RESOLVED
Closed: 5 years ago
status-firefox71:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla71
|
||
Updated•5 years ago
|
Assignee: nobody → bbouvier
You need to log in
before you can comment on or make changes to this bug.
Description
•