[PATCH] D85062: [WebAssembly] GC constructor functions in otherwise unused archive objects

[Dan Gohman via Phabricator via llvm-commits]

sunfish updated this revision to Diff 282449.
sunfish added a comment.

- Reorganize the code a little so that we don't have to call `mark` multiple times.
- Fix a bug where we weren't considering calls from constructors as keeping other constructors live.
- Add a few more tests.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D85062

Files:
  lld/test/wasm/Inputs/ctor-ctor.s
  lld/test/wasm/Inputs/ctor-lib.s
  lld/test/wasm/Inputs/ctor-setup-call-def.s
  lld/test/wasm/Inputs/ctor-setup.s
  lld/test/wasm/Inputs/ctor-start.s
  lld/test/wasm/ctor-gc-setup.test
  lld/test/wasm/ctor-gc.test
  lld/test/wasm/ctor-no-gc.test
  lld/wasm/InputFiles.h
  lld/wasm/MarkLive.cpp
  lld/wasm/Symbols.cpp
  lld/wasm/Writer.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D85062.282449.patch
Type: text/x-patch
Size: 11163 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200802/df51a3c8/attachment-0001.bin>