Closed
Bug 1624490
Opened 4 years ago
Closed 4 years ago
Allow declared segments to contain ref.null
Categories
(Core :: JavaScript: WebAssembly, defect, P3)
Core
JavaScript: WebAssembly
Tracking
()
RESOLVED
FIXED
mozilla76
| Tracking | Status | |
|---|---|---|
| firefox76 | --- | fixed |
People
(Reporter: rhunt, Assigned: rhunt)
References
Details
Attachments
(1 file)
The spec allows for declared segments to contain null [1], but we don't allow this. I believe the reason I didn't was that I couldn't see a use-case for it, but I can't really see any argument against allowing it either.
[1] https://webassembly.github.io/reference-types/core/valid/modules.html#element-segments
|
Assignee | |
Comment 1•4 years ago
|
||
Pushed by rhunt@eqrion.net: https://hg.mozilla.org/integration/autoland/rev/b89a4862242c Allow ref.null in declared segments. r=lth
|
||
Comment 3•4 years ago
|
||
Backed out changeset b89a4862242c (bug 1624490) for causing CompileError asertion failures
https://hg.mozilla.org/integration/autoland/rev/cdf202c2682eccdb9781145451cb707ec157db9d
push that caused the backout: https://treeherder.mozilla.org/#/jobs?repo=autoland&selectedJob=294699981&resultStatus=testfailed%2Cbusted%2Cexception%2Cretry%2Cusercancel%2Crunnable&revision=b89a4862242cf93166205e15ccdff8e9c04de9b5&failure_classification_id=2
Flags: needinfo?(rhunt)
|
|
Assignee | |
Comment 4•4 years ago
|
||
Flags: needinfo?(rhunt)
Pushed by rhunt@eqrion.net: https://hg.mozilla.org/integration/autoland/rev/f8086519b208 Allow ref.null in declared segments. r=lth
|
||
Comment 6•4 years ago
|
||
| bugherder | ||
Status: NEW → RESOLVED
Closed: 4 years ago
status-firefox76:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla76
You need to log in
before you can comment on or make changes to this bug.
Description
•