[PATCH] D123301: [lld-macho][nfc] Factor out & micro-optimize findSymbolAtOffset
Jez Ng via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Apr 7 06:11:30 PDT 2022
int3 marked an inline comment as done.
int3 added inline comments.
================
Comment at: lld/MachO/InputFiles.cpp:1020
- assert(referentIsec->wasCoalesced);
- ++it;
- continue;
----------------
int3 wrote:
> thakis wrote:
> > Previously we advanced it instead of doing `it = isec->relocs.erase(it)` if nothing was found. The rhs doesn't do that. Is that intentional?
> oops, good catch
I think erasing it might actually make more sense, but I'll revisit this in a future diff. this one is supposed to be a NFC
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D123301/new/
https://reviews.llvm.org/D123301
More information about the llvm-commits
mailing list