[PATCH] D59337: [WebAssembly] Error on R_WASM_MEMORY_ADDR relocations against undefined data symbols.

[Sam Clegg via Phabricator via llvm-commits]

sbc100 added inline comments.


================
Comment at: lld/wasm/Writer.cpp:1038
   uint32_t TableIndex = TableBase;
   auto HandleRelocs = [&](InputChunk *Chunk) {
     if (!Chunk->Live)
----------------
ruiu wrote:
> sbc100 wrote:
> > ruiu wrote:
> > > This function is getting longer. I wonder if you can move this lambda out of this function.
> > Ok to do that in a follow up?
> I'd personally like to do beforehand so that the code is cleaner at any point of time, but a follow-up patch is OK.
OK I did with this CL.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D59337