Closed
Bug 1531731
Opened 7 years ago
Closed 7 years ago
WasmLoadRef and WasmStoreRef must be known to alias analysis
Categories
(Core :: JavaScript Engine: JIT, enhancement, P1)
Core
JavaScript Engine: JIT
Tracking
()
RESOLVED
FIXED
mozilla67
| Tracking | Status | |
|---|---|---|
| firefox67 | --- | fixed |
People
(Reporter: lth, Assigned: lth)
Details
Attachments
(1 file)
When I landed MWasmLoadRef and MWasmStoreRef in bug 1508559 (several patches) I made MWasmLoadRef and MWasmStoreRef play nice with alias analysis by introducing an alias class for wasm table slots separate from wasm globals. But I did not know that these MIR nodes must be registered with the GetObject() operation in AliasAnalysis.cpp.
|
Assignee | |
Comment 1•7 years ago
|
||
These new nodes participate in alias analysis and so must be made known to the alias
analysis framework.
Pushed by lhansen@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/74ab0e9aebf5
Register MWasmLoadRef and MWasmStoreRef with AliasAnalysis. r=bbouvier
|
|
||
Comment 3•7 years ago
|
||
| bugherder | ||
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
status-firefox67:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla67
You need to log in
before you can comment on or make changes to this bug.
Description
•