[all-commits] [llvm/llvm-project] 9803cf: [LLD] [COFF] Fix a condition that was missed in 7f...

Martin Storsjö via All-commits all-commits at lists.llvm.org
Thu Oct 15 08:34:28 PDT 2020


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 9803cf57d6fe5c7189255ad74b289c2a0d4a8a6a
      https://github.com/llvm/llvm-project/commit/9803cf57d6fe5c7189255ad74b289c2a0d4a8a6a
  Author: Martin Storsjö <martin at martin.st>
  Date:   2020-10-15 (Thu, 15 Oct 2020)

  Changed paths:
    M lld/COFF/SymbolTable.cpp

  Log Message:
  -----------
  [LLD] [COFF] Fix a condition that was missed in 7f0e6c31c255. NFC.

This should fix cases when e.g. auto import is enabled without
mingw mode in total being enabled.

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


  Commit: a012c704b5e5b60f9d2a7304d27cbc84a3619571
      https://github.com/llvm/llvm-project/commit/a012c704b5e5b60f9d2a7304d27cbc84a3619571
  Author: Martin Storsjö <martin at martin.st>
  Date:   2020-10-15 (Thu, 15 Oct 2020)

  Changed paths:
    M lld/COFF/Driver.cpp
    M lld/COFF/InputFiles.h
    M lld/COFF/LTO.cpp
    M lld/COFF/MinGW.cpp
    M lld/COFF/MinGW.h
    M lld/COFF/Options.td
    M lld/COFF/SymbolTable.cpp
    M lld/COFF/Symbols.h
    M lld/MinGW/Driver.cpp
    M lld/MinGW/Options.td
    A lld/test/COFF/wrap-i386.s
    A lld/test/COFF/wrap-import.ll
    A lld/test/COFF/wrap-lto-1.ll
    A lld/test/COFF/wrap-lto-2.ll
    A lld/test/COFF/wrap-real-missing.s
    A lld/test/COFF/wrap-with-archive.s
    A lld/test/COFF/wrap.s
    M lld/test/MinGW/driver.test

  Log Message:
  -----------
  [LLD] [COFF] Implement a GNU/ELF like -wrap option

Add a simple forwarding option in the MinGW frontend, and implement
the private -wrap option in the COFF linker.

The feature in lld-link isn't gated by the -lldmingw option, but
the option is left as a private, undocumented option primarily
used by the MinGW driver.

The implementation is significantly based on the support for --wrap
in the ELF linker, but many small nuance details are different
between the ELF and COFF linkers, ending up with more than a few
implementation differences.

This fixes https://bugs.llvm.org/show_bug.cgi?id=47384.

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


Compare: https://github.com/llvm/llvm-project/compare/7c6bfd90ab2d...a012c704b5e5


More information about the All-commits mailing list