[PATCH] D56294: [ObjectYAML] [COFF] Support multiple symbols with the same name

[Martin Storsjö via Phabricator via llvm-commits]

mstorsjo marked 2 inline comments as done.
mstorsjo added inline comments.


================
Comment at: tools/obj2yaml/coff2yaml.cpp:151
+    Obj.getSymbolName(Symbol, Name);
+    auto It = SymbolUnique.find(Name);
+    if (It == SymbolUnique.end())
----------------
alexshap wrote:
> side note: double lookup (1st - find, second - operator []) can be avoided here if we switch to using the method "insert" instead (it returns the corresponding iterator).
Oh, nice, will do.


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

https://reviews.llvm.org/D56294