diff options
author | Alon Zakai <alonzakai@gmail.com> | 2016-08-22 16:01:10 -0700 |
---|---|---|
committer | Alon Zakai <alonzakai@gmail.com> | 2016-09-07 09:55:05 -0700 |
commit | f3bb9debe6af0576d76bda1580df2570e749bd36 (patch) | |
tree | ac4bc7b6e7dfe89cb5c1df7ecb44833e83d313bc /src | |
parent | 83e0dde16f381b28c1ee0d099de25444b0f34e58 (diff) | |
download | binaryen-f3bb9debe6af0576d76bda1580df2570e749bd36.tar.gz binaryen-f3bb9debe6af0576d76bda1580df2570e749bd36.tar.bz2 binaryen-f3bb9debe6af0576d76bda1580df2570e749bd36.zip |
handle asm.js globals that are set and the return value used
Diffstat (limited to 'src')
-rw-r--r-- | src/asm2wasm.h | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/src/asm2wasm.h b/src/asm2wasm.h index 52598ad0b..321d5275e 100644 --- a/src/asm2wasm.h +++ b/src/asm2wasm.h @@ -1014,7 +1014,10 @@ Function* Asm2WasmBuilder::processFunction(Ref ast) { } // global var assert(mappedGlobals.find(name) != mappedGlobals.end()); - return builder.makeSetGlobal(name, process(ast[3])); + auto* ret = builder.makeSetGlobal(name, process(ast[3])); + // set_global does not return; if our value is trivially not used, don't emit a load (if nontrivially not used, opts get it later) + if (astStackHelper.getParent()[0] == STAT) return ret; + return builder.makeSequence(ret, builder.makeGetGlobal(name, ret->value->type)); } else if (ast[2][0] == SUB) { Ref target = ast[2]; assert(target[1][0] == NAME); |