diff options
| author | Jez Ng <jezng@fb.com> | 2022-04-11 15:45:25 -0400 |
|---|---|---|
| committer | Jez Ng <jezng@fb.com> | 2022-04-11 15:45:46 -0400 |
| commit | 1cff723ff527030295d84d3bb52d4d0b7cddb43a (patch) | |
| tree | c3ac0d6271599a707687fcc21abd9e697a07f5c4 /clang/lib/Basic/FileManager.cpp | |
| parent | a2b212bafc9508e54f94b1dce7a46e21ff9b5e91 (diff) | |
| download | llvm-1cff723ff527030295d84d3bb52d4d0b7cddb43a.zip llvm-1cff723ff527030295d84d3bb52d4d0b7cddb43a.tar.gz llvm-1cff723ff527030295d84d3bb52d4d0b7cddb43a.tar.bz2 | |
[lld-macho][nfc] Use includeInSymtab for all symtab-skipping logic
{D123302} got me looking deeper at `includeInSymtab`. I thought it was a
little odd that there were excluded (live) symbols for which
`includeInSymtab` was false; we shouldn't have so many different ways to
exclude a symbol. As such, this diff makes the `L`-prefixed-symbol
exclusion code use `includeInSymtab` too. (Note that as part of our
support for `__eh_frame`, we will also be excluding all `__eh_frame`
symbols from the symtab in a future diff.)
Another thing I noticed is that the `emitStabs` code never has to deal
with excluded symbols because `SymtabSection::finalize()` already
filters them out. As such, I've updated the comments and asserts from
{D123302} to reflect this.
Reviewed By: #lld-macho, thakis
Differential Revision: https://reviews.llvm.org/D123433
Diffstat (limited to 'clang/lib/Basic/FileManager.cpp')
0 files changed, 0 insertions, 0 deletions