[PATCH] D121420: [lld][WebAssembly] Take advantage of extended const expression when available

Mon Mar 14 08:46:50 PDT 2022

sbc100 added a comment.

In D121420#3379498 <https://reviews.llvm.org/D121420#3379498>, @dschuff wrote:

> Would there still be an advantage to combining the segments? e.g. would the library load faster if there were only one?

We already combine segments of the same type/prefix unless `--no-merge-data-segments` is passed... so in the normal case there are just a fixed number or possible output segments: `.data` `.bss` `.tdata` `.tbss`, plus any user-defined segments.    This later-phase combining was always just a stop gap to handler the lack of extended const expressions.  See the description of https://reviews.llvm.org/D96453 for why we delay the merging in this case.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D121420