[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