[PATCH] D118916: [lld-macho][nfc] Minor clean up

Shoaib Meenai via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 3 09:00:16 PST 2022


smeenai added a comment.

I'm wary about using linked lists, especially for relocations ... could you measure the perf and memory usage difference from that? (Maybe each subsection has few enough relocations that it doesn't make a massive difference, but I'm still curious.)

If pointer stability is the main motivation, would `std::deque` work?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D118916/new/

https://reviews.llvm.org/D118916



More information about the llvm-commits mailing list