WasmLoadRef and WasmStoreRef must be known to alias analysis

RESOLVED FIXED in Firefox 67

Status

()

enhancement
P1
normal
RESOLVED FIXED
6 months ago
6 months ago

People

(Reporter: lth, Assigned: lth)

Tracking

unspecified
mozilla67
Points:
---

Firefox Tracking Flags

(firefox67 fixed)

Details

Attachments

(1 attachment)

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.

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
Status: ASSIGNED → RESOLVED
Closed: 6 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla67
You need to log in before you can comment on or make changes to this bug.