[PATCH] D69607: Add a feature to explain why some file gets included to the linker's output

[Rui Ueyama via Phabricator via llvm-commits]

ruiu marked 5 inline comments as done.
ruiu added inline comments.


================
Comment at: lld/ELF/Explain.cpp:157
+
+  if (!seen.insert(sym.file).second)
+    return;
----------------
MaskRay wrote:
> We can make edges a superset of specialEdges, then we can just delete `seen` and use `edges.try_emplace(...)` here.
We probably could but looks like it also makes sense to keep them distinctive sets, so I'm leaning towards not doing that.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D69607