[PATCH] D118155: [OpenMP] Improve symbol resolution for OpenMP Offloading LTO

Shilei Tian via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jan 31 19:55:37 PST 2022


tianshilei1992 added a comment.

LG with one nit.



================
Comment at: clang/tools/clang-linker-wrapper/ClangLinkerWrapper.cpp:777-781
+        // Record if we've seen these symbols in any object or shared libraries.
+        if ((*ObjFile)->isRelocatableObject()) {
+          UsedInRegularObj[*Name] = true;
+        } else
+          UsedInSharedLib[*Name] = true;
----------------



Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D118155



More information about the cfe-commits mailing list