[all-commits] [llvm/llvm-project] d2a0ef: [lld][WebAssembly] Don't force the export symbols ...

Sam Clegg via All-commits all-commits at lists.llvm.org
Tue Feb 15 17:30:34 PST 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: d2a0ef984470536cf3fbdef1ee66e6fe113501b4
      https://github.com/llvm/llvm-project/commit/d2a0ef984470536cf3fbdef1ee66e6fe113501b4
  Author: Sam Clegg <sbc at chromium.org>
  Date:   2022-02-15 (Tue, 15 Feb 2022)

  Changed paths:
    M lld/test/wasm/tls-non-shared-memory.s
    M lld/wasm/Symbols.cpp
    M lld/wasm/SyntheticSections.cpp

  Log Message:
  -----------
  [lld][WebAssembly] Don't force the export symbols assiged internal/dummy GOT entries

Symbols with regular GOT entries do need to be exported, but those that
are internalized (and have dymmy/internal GOT entries) need not be
exported.

This happens to fix the failures on the emscripten waterfall where extra
symbols were being exported by the linker (and then later removed by
wasm-opt).

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




More information about the All-commits mailing list