1637418 - Change ref.func global initializers to declare function as valid ref.func target

Status: RESOLVED FIXED

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

Description

[Ryan Hunt [:rhunt] (on leave until early May)]

This is issue#76 [1] and was discussed in CG meeting 05-12-20 where we resolved to treat all ref.func instruction in the 'module environment' as a declaration that a function index is a valid ref.func target in the code section.

[1] https://github.com/WebAssembly/reference-types/issues/76

Comment 1

[Ryan Hunt [:rhunt] (on leave until early May)]

Attachment: Bug 1637418 - Allow whole module environment to declare valid ref.func targets. r?lth

The CG agreed to resolve reference-types#76 by treating all ocurrences of ref.func
in the 'module environment' as declaring a valid ref.func target for the code
section. Currently the only place we don't do this is in the global section.

[1] https://github.com/WebAssembly/reference-types/issues/76

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/39a3f904ac83
Allow whole module environment to declare valid ref.func targets. r=lth

Comment 3

[Cristian Brindusan [:cbrindusan]]

https://hg.mozilla.org/mozilla-central/rev/39a3f904ac83