[PATCH] D62832: [WebAssembly] add memory map file support for wasm-ld

Mon Jun 3 18:27:14 PDT 2019

ruiu added a comment.

I'd rename this patch to avoid "memory map file" because that term is overloaded and can be interpreted in a different way. I'd say implementing -print-map.

================
Comment at: lld/wasm/MapFile.cpp:14
+//   Address     Size Align Symbol
+//       400      fa0     1 p
+//      13a0        4     1 q
----------------
This format is different from lld/ELF. I think we have a column for input files, input sections and output sections. Is this an intentional change?

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D62832