[PATCH] D121349: [WebAssembly] Second phase of implementing extended const proposal.

Mon Mar 14 14:37:16 PDT 2022

sbc100 added a comment.

In D121349#3380735 <https://reviews.llvm.org/D121349#3380735>, @saugustine wrote:

> This patch triggers a bunch of uninitialized memory under msan errors when running llvm-project/lld/test/wasm:data-layout.s (and several others) as below:
>
>   exit status 2
>   ==1915==WARNING: MemorySanitizer: use-of-uninitialized-value
>       #0 0x55e0ff9ca683 in (anonymous namespace)::WasmDumper::dump() /llvm-project/llvm/tools/obj2yaml/wasm2yaml.cpp:374:13
>       #1 0x55e0ff9c1b87 in wasm2yaml(llvm::raw_ostream&, llvm::object::WasmObjectFile const&) llvm-project/llvm/tools/obj2yaml/wasm2yaml.cpp:411:50
>       #2 0x55e0ff9be3c9 in dumpObject llvm-project/llvm/tools/obj2yaml/obj2yaml.cpp:40:29
>       #3 0x55e0ff9be3c9 in dumpInput llvm-project/llvm/tools/obj2yaml/obj2yaml.cpp:67:12
>       #4 0x55e0ff9be3c9 in main llvm-project/llvm/tools/obj2yaml/obj2yaml.cpp:89:19
>       #5 0x7f7a5befc8d2 in __libc_start_main (/usr/grte/v5/lib64/libc.so.6+0x628d2) (BuildId: 7cfed7708e5ab7fcb286b373de21ee76)
>       #6 0x55e0ff798429 in _start /usr/grte/v5/debug-src/src/csu/../sysdeps/x86_64/start.S:120
>   
>   SUMMARY: MemorySanitizer: use-of-uninitialized-value llvm-project/llvm/tools/obj2yaml/wasm2yaml.cpp:374:13 in (anonymous namespace)::WasmDumper::dump()
>   Exiting
>   FileCheck error: '<stdin>' is empty.
>   FileCheck command line:  llvm-project/llvm/FileCheck --allow-unused-prefixes llvm-project/lld/test/wasm/data-layout.s -check-prefix=CHECK-SHARED

Thanks, investigating.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D121349