1902835 - Don't serialize modules compiled with builtins

Status: ASSIGNED

(Core :: JavaScript: WebAssembly, defect, P2)

Description

[Ryan Hunt [:rhunt]]

If a module is compile with js-string-builtins, we need to be careful to not serialize/deserialize as our caching system doesn't have a way for us to key the alternative data cache off of the requested builtins.

Comment 1

[Ryan Hunt [:rhunt]]

Attachment: Bug 1902835 - wasm: Don't serialize if there are builtin modules. r?yury

Builtin modules change the JIT code. A future fetch + compile of a module
may choose to not use builtin modules, so the cached module we store in
the alt-data of a fetch is only sometimes valid. We would need to validate
the cached version's builtin modules matches the new requested compile's
options. Our system doesn't have an easy way to do that right now, so
just disable caching of these modules for now.

Depends on D213866