[ELF] Correct error message when OUTPUT_FORMAT is used
authorShoaib Meenai <smeenai@fb.com>
Thu, 12 Mar 2020 22:25:36 +0000 (15:25 -0700)
committerShoaib Meenai <smeenai@fb.com>
Fri, 13 Mar 2020 05:54:53 +0000 (22:54 -0700)
commit2822852ffc4649b09670e6f287871990536e3c7b
tree6d2008cfabe02e917b54376e48970ba43cb40218
parent1ba3d2639d1c489a3876f72d822446c10fca0a23
[ELF] Correct error message when OUTPUT_FORMAT is used

Any OUTPUT_FORMAT in a linker script overrides the emulation passed on
the command line, so record the passed bfdname and use that in the error
message about incompatible input files.

This prevents confusing error messages. For example, if you explicitly
pass `-m elf_x86_64` to LLD but accidentally include a linker script
which sets `OUTPUT_FORMAT(elf32-i386)`, LLD would previously complain
about your input files being compatible with elf_x86_64, which isn't the
actual issue, and is confusing because the input files are in fact
x86-64 ELF files.

Interestingly enough, this also prevents a segfault! When we don't pass
`-m` and we have an object file which is incompatible with the
`OUTPUT_FORMAT` set by a linker script, the object file is checked for
compatibility before it's added to the objectFiles vector.
config->emulation, objectFiles, and sharedFiles will all be empty, so
we'll attempt to access bitcodeFiles[0], but bitcodeFiles is also empty,
so we'll segfault. This commit prevents the segfault by adding
OUTPUT_FORMAT as a possible source of machine configuration, and it also
adds an llvm_unreachable to diagnose similar issues in the future.

Differential Revision: https://reviews.llvm.org/D76109
lld/ELF/Config.h
lld/ELF/InputFiles.cpp
lld/ELF/ScriptParser.cpp
lld/test/ELF/incompatible.s