T0b1/wasmtime
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fix comments about bulk memory that incorrectly referenced "multi-value"

Browse Source
This commit is contained in:
Nick Fitzgerald
2020-02-25 10:14:59 -08:00
parent aec8cc1904
commit 39307b2b36
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
crates/runtime/src/instance.rs
View File

@@ -1199,9 +1199,9 @@ fn initialize_tables(instance: &Instance) -> Result<(), InstantiationError> {
let anyfunc = instance.get_caller_checked_anyfunc(*func_idx); let anyfunc = instance.get_caller_checked_anyfunc(*func_idx);
table table
.set(u32::try_from(start + i).unwrap(), anyfunc) .set(u32::try_from(start + i).unwrap(), anyfunc)
// Note that when multi-value is disabled, this will never fail // Note that when bulk memory is disabled, this will never fail
// since we bounds check table element initialization before // since we bounds check table element initialization before
// doing any table slot writes. However when multi-value is // doing any table slot writes. However when bulk memory is
// enabled, these become runtime traps and the intermediate // enabled, these become runtime traps and the intermediate
// table slot writes are visible. // table slot writes are visible.
.map_err(|()| { .map_err(|()| {