| commit | 4791a668f573a71da97e82882fbcecade7533d42 | [log] [tgz] |
|---|---|---|
| author | Sam Clegg <sbc@chromium.org> | Tue Nov 20 19:25:07 2018 +0000 |
| committer | Sam Clegg <sbc@chromium.org> | Tue Nov 20 19:25:07 2018 +0000 |
| tree | 1d2656fbb20d03122618d85fc80c856f58e1c631 | |
| parent | 377748fd7bbf5798fe54a032e79d61c6feb86ccb [diff] |
[WebAssembly] WebAssemblyLowerEmscriptenEHSjLj: use getter/setter for accessing tempRet0 Rather than assuming that `tempRet0` exists in linear memory only assume the getter/setter functions exist. This avoids conflicting with binaryen which declares a wasm global for this purpose and defines it's own getter and setter for that. The other advantage of doing things this way is that it leaving it up to the linker/finalizer to decide how to actually store this temporary. As it happens binaryen uses a wasm global which is more appropriate since it is thread safe. This also allows us to change the way this is stored in the future (memory, TLS memory, wasm global) without modifying LLVM. This is part of a 4 part change: LLVM: https://reviews.llvm.org/D53240 fastcomp: https://github.com/kripken/emscripten-fastcomp/pull/237 emscripten: https://github.com/kripken/emscripten/pull/7358 binaryen: https://github.com/WebAssembly/binaryen/pull/1709 Differential Revision: https://reviews.llvm.org/D53240 llvm-svn: 347340