[PATCH] D130787: [lld][ELF] Rename SymbolTable::symbols() to SymbolTable::getSymbols(). NFC
Sam Clegg via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Aug 5 12:00:29 PDT 2022
sbc100 added a comment.
In D130787#3687966 <https://reviews.llvm.org/D130787#3687966>, @sbc100 wrote:
> For example, see the existing `InputFile::getSymbols` which exists in all 3 backends
Do you feel strongly about `.symbols()` over `.getSymbols()`? This change seems like the least intrusive way to consistency, and I would rather land this, but if you feel strongly we can move other methods/backends over to `.symbols()` instead.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D130787/new/
https://reviews.llvm.org/D130787
More information about the llvm-commits
mailing list