summaryrefslogtreecommitdiff
path: root/lld/wasm/Driver.cpp
diff options
context:
space:
mode:
authorRui Ueyama <ruiu@google.com>2018-01-12 22:09:19 +0000
committerRui Ueyama <ruiu@google.com>2018-01-12 22:09:19 +0000
commit2f1aa3783694bb35d75fee2b1cdcab68a8356547 (patch)
treef4550db52d83995d79cd4bc688d93843200c9eb4 /lld/wasm/Driver.cpp
parent0ae1272d3e1c9cca99cf3ef2d25b76b072cd5d74 (diff)
Allow unaligned access to ELF file data structures.
The ELF specification says that all ELF data structures are aligned to their natural alignments both in memory and file. That means when we access mmap'ed ELF files, we could assume that all data structures are aligned properly. However, in reality, we assume that the data structures are aligned only to two bytes because .a files only guarantee that their member files are aligned to two bytes in archive files. So the data access is already unaligned. This patch relaxes the alignment requirement even more, so that we accept unaligned access to all ELF data structures. This patch in particular makes lld bug-compatible with icc. Intel C compiler doesn't seem to care about data alignment and generates unaligned relocation sections (https://bugs.llvm.org/show_bug.cgi?id=35854). I also saw another instance of compatibility issues with our internal tool which creates unaligned section headers. Because GNU linkers are not picky about alignment, looks like it is not uncommon that ELF-generating tools create unaligned files. There is a performance penalty with this patch on host machines on which unaligned access is expensive. x86 and AArch64 are fine. ARMv6 is a problem, but I don't think using ARMv6 machines as hosts is common, so I believe it's not a real problem. Differential Revision: https://reviews.llvm.org/D41978
Diffstat (limited to 'lld/wasm/Driver.cpp')
0 files changed, 0 insertions, 0 deletions