[PATCH] D63344: lld/elf: Deduplicate undefined symbol diagnostics

[Fangrui Song via Phabricator via llvm-commits]

MaskRay added inline comments.


================
Comment at: lld/ELF/Relocations.cpp:789
+  DenseMap<Symbol *, UndefinedDiag *> FirstRef;
+  for (size_t I = 0; I < Undefs.size(); ++I) {
+    UndefinedDiag &Undef = Undefs[I];
----------------
I think this loop can be changed to range-based


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

https://reviews.llvm.org/D63344