[PATCH] D117412: [lld][WebAssembly] Perform data relocations during start function

Sam Clegg via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 19 13:28:54 PST 2022


sbc100 added a comment.

In D117412#3256093 <https://reviews.llvm.org/D117412#3256093>, @dschuff wrote:

> This change (probably) broke the SAFE_HEAP tests on the emscripten waterfall: https://logs.chromium.org/logs/emscripten-releases/buildbucket/cr-buildbucket/8824622360264175889/+/u/Emscripten_testsuite__SAFE_HEAP_/stdout
> There are other changes in the LLVM roll (https://chromium.googlesource.com/emscripten-releases/+/cd22f208c7dda41d695549e17f8ddc4ae533769d) but this is the only wasm-related one.

Yup.. I noticed that too.  Fix is in https://github.com/WebAssembly/binaryen/pull/4463


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D117412/new/

https://reviews.llvm.org/D117412



More information about the llvm-commits mailing list