[all-commits] [llvm/llvm-project] 704e0b: [lld][WebAssembly] Honor --allow-undefined for dat...

[Sam Clegg via All-commits]

  Branch: refs/heads/allow_undefined_data
  Home:   https://github.com/llvm/llvm-project
  Commit: 704e0ba53cb2d849fef08d1850175de8efd814c7
      https://github.com/llvm/llvm-project/commit/704e0ba53cb2d849fef08d1850175de8efd814c7
  Author: Sam Clegg <sbc at chromium.org>
  Date:   2020-05-01 (Fri, 01 May 2020)

  Changed paths:
    M lld/test/wasm/undefined-data.ll
    M lld/wasm/Relocations.cpp

  Log Message:
  -----------
  [lld][WebAssembly] Honor --allow-undefined for data symbols too

This was originally the way this worked before before
https://reviews.llvm.org/D60882.

In retrospect it seems inconsistent that `--allow-undefined` doesn't
work for all symbols.  See:
https://groups.google.com/g/emscripten-discuss/c/HSRgQiIq1gI/m/Kt9oFWHiAwAJ

I'm also planning a followup change which implement the full
`--unresolved-symbols=..` flags supported by ELF linkers (both ld and
ld.lld) since it seems more standard.