[PATCH] D145263: [lld-macho] Remove duplicate minimum version info

[Keith Smiley via Phabricator via llvm-commits]

keith added inline comments.


================
Comment at: lld/MachO/Driver.cpp:688
+parsePlatformVersions(const ArgList &args) {
   std::map<PlatformType, PlatformVersion> platformVersions;
   const PlatformVersion *lastVersionInfo = nullptr;
----------------
int3 wrote:
> while we're here, could we change this to a `DenseMap` or `IndexedMap`?
> 
> Doesn't really matter for perf but we try to avoid `std::map` in general. And this might save some binary size
it looks like both of those have new requirements for the key type, I will do it in a separate change to avoid noise in this diff


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D145263