ELF: Rename error -> fatal and redefine error as a non-noreturn function.
authorRui Ueyama <ruiu@google.com>
Thu, 28 Jan 2016 18:40:06 +0000 (18:40 +0000)
committerRui Ueyama <ruiu@google.com>
Thu, 28 Jan 2016 18:40:06 +0000 (18:40 +0000)
commit64cfffd333b261ebb68a6ee04b09cc42d32c1992
tree3320a40b26ea9552b91802bbf4b83767398f1f1e
parentec8d086c64f4012714405fb0d7b0729a33c18179
ELF: Rename error -> fatal and redefine error as a non-noreturn function.

In many situations, we don't want to exit at the first error even in the
process model. For example, it is better to report all undefined symbols
rather than reporting the first one that the linker picked up randomly.

In order to handle such errors, we don't need to wrap everything with
ErrorOr (thanks for David Blaikie for pointing this out!) Instead, we
can set a flag to record the fact that we found an error and keep it
going until it reaches a reasonable checkpoint.

This idea should be applicable to other places. For example, we can
ignore broken relocations and check for errors after visiting all relocs.

In this patch, I rename error to fatal, and introduce another version of
error which doesn't call exit. That function instead sets HasError to true.
Once HasError becomes true, it stays true, so that we know that there
was an error if it is true.

I think introducing a non-noreturn error reporting function is by itself
a good idea, and it looks to me that this also provides a gradual path
towards lld-as-a-library (or at least embed-lld-to-your-program) without
sacrificing code readability with lots of ErrorOr's.

http://reviews.llvm.org/D16641

llvm-svn: 259069
14 files changed:
lld/ELF/Driver.cpp
lld/ELF/Driver.h
lld/ELF/DriverUtils.cpp
lld/ELF/Error.cpp
lld/ELF/Error.h
lld/ELF/InputFiles.cpp
lld/ELF/InputSection.cpp
lld/ELF/LinkerScript.cpp
lld/ELF/OutputSections.cpp
lld/ELF/SymbolTable.cpp
lld/ELF/Target.cpp
lld/ELF/Writer.cpp
lld/include/lld/Driver/Driver.h
lld/lib/Driver/UniversalDriver.cpp