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

Jez Ng via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 3 09:48:19 PST 2022


int3 added a comment.

The list change is def a bit more controversial. I would split out the map changes into its own diff :)

Also, could you elaborate on where you anticipate needing pointer stability?

Finally, if we really need a list, we might want to use LLVM's ilist for better locality.


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