diff options
| author | Georgii Rymar <grimar@accesssoftek.com> | 2019-12-03 15:48:29 +0300 |
|---|---|---|
| committer | Georgii Rymar <grimar@accesssoftek.com> | 2019-12-04 13:12:02 +0300 |
| commit | daff7b85890b31085f75b8e9694e074745518069 (patch) | |
| tree | 4f3bf65be657c6913c8beb3971e2323b2de371f2 /llvm/lib/Support/CommandLine.cpp | |
| parent | 150c8dd13be4a9d9a9f631a507871359117871ca (diff) | |
| download | llvm-daff7b85890b31085f75b8e9694e074745518069.zip llvm-daff7b85890b31085f75b8e9694e074745518069.tar.gz llvm-daff7b85890b31085f75b8e9694e074745518069.tar.bz2 | |
[yaml2obj] - Make DynamicSymbols to be Optional<> too.
We already have Symbols property to list regular symbols and
it is currently Optional<>. This patch makes DynamicSymbols to be optional
too. With this there is no need to define a dummy symbol anymore to trigger
creation of the .dynsym and it is now possible to define an empty .dynsym using
just the following line:
DynamicSymbols: []
(it is important to have when you do not want to have dynamic symbols,
but want to have a .dynsym)
Now the code is consistent and it helped to fix a bug: previously we
did not report an error when both Content/Size and an empty
Symbols/DynamicSymbols list were specified.
Differential revision: https://reviews.llvm.org/D70956
Diffstat (limited to 'llvm/lib/Support/CommandLine.cpp')
0 files changed, 0 insertions, 0 deletions