[all-commits] [llvm/llvm-project] 57776f: [ELF] Fix lld build on Windows/MinGW
Ayke via All-commits
all-commits at lists.llvm.org
Sat Nov 16 04:41:54 PST 2019
Branch: refs/heads/master
Home: https://github.com/llvm/llvm-project
Commit: 57776f71fa32a5b170a9ce82cb2c2da0a207908c
https://github.com/llvm/llvm-project/commit/57776f71fa32a5b170a9ce82cb2c2da0a207908c
Author: Ayke van Laethem <aykevanlaethem at gmail.com>
Date: 2019-11-16 (Sat, 16 Nov 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
More information about the All-commits
mailing list