[all-commits] [llvm/llvm-project] bd4812: [WebAssembly] Use llvm::Optional to store optional...

[Sam Clegg via All-commits]

  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: bd4812776bc73ca27bf6c68629a20f03268cdd6e
      https://github.com/llvm/llvm-project/commit/bd4812776bc73ca27bf6c68629a20f03268cdd6e
  Author: Sam Clegg <sbc at chromium.org>
  Date:   2020-02-19 (Wed, 19 Feb 2020)

  Changed paths:
    M lld/wasm/Driver.cpp
    M lld/wasm/InputFiles.cpp
    M lld/wasm/LTO.cpp
    M lld/wasm/SymbolTable.cpp
    M lld/wasm/SymbolTable.h
    M lld/wasm/Symbols.h
    M lld/wasm/SyntheticSections.cpp
    M llvm/include/llvm/BinaryFormat/Wasm.h
    M llvm/include/llvm/MC/MCSymbolWasm.h
    M llvm/lib/Object/WasmObjectFile.cpp
    M llvm/test/MC/WebAssembly/debug-info.ll
    M llvm/test/tools/llvm-readobj/wasm/symbols.test
    M llvm/test/tools/llvm-readobj/wasm/wasm-imports.test
    M llvm/tools/llvm-readobj/WasmDumper.cpp

  Log Message:
  -----------
  [WebAssembly] Use llvm::Optional to store optional symbol attributes.  NFC.

The changes the in-memory representation of wasm symbols such that their
optional ImportName and ImportModule use llvm::Optional.

ImportName is set whenever WASM_SYMBOL_EXPLICIT_NAME flag is set.
ImportModule (for imports) is currently always set since it defaults to
"env".

In the future we can possibly extent to binary format distingish
import which have explit module names.

Tags: #llvm

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