[Openmp-commits] [PATCH] D142484: [LinkerWrapper] Only import static libraries with needed symbols

[Artem Belevich via Phabricator via Openmp-commits]

tra added a subscriber: MaskRay.
tra added a comment.

@MaskRay - we seem to be reinventing the linker here and could use your expertise.



================
Comment at: clang/tools/clang-linker-wrapper/ClangLinkerWrapper.cpp:1217-1218
+///   1) It defines an undefined symbol in a regular object filie.
+///   2) It defines a global symbol without hidden visibility that has not
+///      yet been defined.
+Expected<bool> getSymbols(StringRef Image, StringSaver &Saver,
----------------
How do we handle conflicting symbols defined more than once? 



Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D142484