[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