[WebAssembly] Add a flag to control merging data segments
authorSam Clegg <sbc@chromium.org>
Thu, 10 May 2018 18:23:51 +0000 (18:23 +0000)
committerSam Clegg <sbc@chromium.org>
Thu, 10 May 2018 18:23:51 +0000 (18:23 +0000)
commit668447677a8bac515dbdf1f8ad2479819d135f45
tree82cede3661323aa7e8edd3310f4bb41a22ca59ab
parentcac2b3349ede9faf34c8f8fd2553a036c701c22b
[WebAssembly] Add a flag to control merging data segments

Merging data segments produces smaller code sizes because each segment
has some boilerplate. Therefore, merging data segments is generally the
right approach, especially with wasm where binaries are typically
delivered over the network.

However, when analyzing wasm binaries, it can be helpful to get a
conservative picture of which functions are using which data
segments[0]. Perhaps there is a large data segment that you didn't
expect to be included in the wasm, introduced by some library you're
using, and you'd like to know which library it was. In this scenario,
merging data segments only makes the analysis worse.

Alternatively, perhaps you will remove some dead functions by-hand[1]
that can't be statically proven dead by the compiler or lld, and
removing these functions might make some data garbage collect-able, and
you'd like to run `--gc-sections` again so that this now-unused data can
be collected. If the segments were originally merged, then a single use
of the merged data segment will entrench all of the data.

[0] https://github.com/rustwasm/twiggy
[1] https://github.com/fitzgen/wasm-snip

Patch by Nick Fitzgerald!

Differential Revision: https://reviews.llvm.org/D46417

llvm-svn: 332013
lld/test/wasm/data-segment-merging.ll [new file with mode: 0644]
lld/wasm/Config.h
lld/wasm/Driver.cpp
lld/wasm/Options.td
lld/wasm/Writer.cpp