[all-commits] [llvm/llvm-project] 0a64fe: [ELF] Fix lld build on Windows/MinGW

Ayke via All-commits all-commits at lists.llvm.org
Tue Dec 3 13:16:41 PST 2019


  Branch: refs/heads/release/9.x
  Home:   https://github.com/llvm/llvm-project
  Commit: 0a64fe568090a6e298669d901cdff7b356194aa5
      https://github.com/llvm/llvm-project/commit/0a64fe568090a6e298669d901cdff7b356194aa5
  Author: Ayke van Laethem <aykevanlaethem at gmail.com>
  Date:   2019-12-03 (Tue, 03 Dec 2019)

  Changed paths:
    M lld/ELF/Symbols.h

  Log Message:
  -----------
  [ELF] Fix lld build on Windows/MinGW

The patch in https://reviews.llvm.org/D64077 causes a build failure
because both the Defined and SharedSymbol classes are bigger than 80
bytes on MinGW 8.

This patch fixes this build failure by changing the type of the
bitfields. It is a similar change to the bitfield changes in
https://reviews.llvm.org/D64238, but instead of changing to bool I
decided to use uint8_t because one of the bitfields takes up two bits
instead of one.

Note: the patch is slightly different from the one reviewed in
Phabricator, but it is a trivial change to align it with LLVM master
instead of LLVM 9. Also, it passes all lld tests.

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

(cherry picked from commit 57776f71fa32a5b170a9ce82cb2c2da0a207908c)




More information about the All-commits mailing list