[PATCH] D57043: [clangd] Use the shared forced-linking code.

[Haojian Wu via Phabricator via cfe-commits]

hokein created this revision.
hokein added a reviewer: sammccall.
Herald added subscribers: kadircet, arphaman, jkorous, MaskRay, ioeric, 
ilya-biryukov, mgorny, srhines.

Also add a missing MPI module.


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D57043

Files:
  clangd/CMakeLists.txt
  clangd/ClangdUnit.cpp


Index: clangd/ClangdUnit.cpp
===================================================================
--- clangd/ClangdUnit.cpp
+++ clangd/ClangdUnit.cpp
@@ -572,29 +572,6 @@
 }
 
 } // namespace clangd
-namespace tidy {
-// Force the linker to link in Clang-tidy modules.
-#define LINK_TIDY_MODULE(X)                                                    
\
-  extern volatile int X##ModuleAnchorSource;                                   
\
-  static int LLVM_ATTRIBUTE_UNUSED X##ModuleAnchorDestination =                
\
-      X##ModuleAnchorSource
-LINK_TIDY_MODULE(CERT);
-LINK_TIDY_MODULE(Abseil);
-LINK_TIDY_MODULE(Boost);
-LINK_TIDY_MODULE(Bugprone);
-LINK_TIDY_MODULE(LLVM);
-LINK_TIDY_MODULE(CppCoreGuidelines);
-LINK_TIDY_MODULE(Fuchsia);
-LINK_TIDY_MODULE(Google);
-LINK_TIDY_MODULE(Android);
-LINK_TIDY_MODULE(Misc);
-LINK_TIDY_MODULE(Modernize);
-LINK_TIDY_MODULE(Performance);
-LINK_TIDY_MODULE(Portability);
-LINK_TIDY_MODULE(Readability);
-LINK_TIDY_MODULE(ObjC);
-LINK_TIDY_MODULE(HICPP);
-LINK_TIDY_MODULE(Zircon);
-#undef LINK_TIDY_MODULE
-} // namespace tidy
 } // namespace clang
+
+#include "../clang-tidy/ClangTidyForceLinker.h"
\ No newline at end of file
Index: clangd/CMakeLists.txt
===================================================================
--- clangd/CMakeLists.txt
+++ clangd/CMakeLists.txt
@@ -107,6 +107,12 @@
   ${CLANGD_ATOMIC_LIB}
   )
 
+if( CLANG_ENABLE_STATIC_ANALYZER )
+  target_link_libraries(clangDaemon PRIVATE
+    clangTidyMPIModule
+  )
+endif()
+
 if( LLVM_LIB_FUZZING_ENGINE OR LLVM_USE_SANITIZE_COVERAGE )
   add_subdirectory(fuzzer)
 endif()



Index: clangd/ClangdUnit.cpp
===================================================================
--- clangd/ClangdUnit.cpp
+++ clangd/ClangdUnit.cpp
@@ -572,29 +572,6 @@
 }
 
 } // namespace clangd
-namespace tidy {
-// Force the linker to link in Clang-tidy modules.
-#define LINK_TIDY_MODULE(X)                                                    \
-  extern volatile int X##ModuleAnchorSource;                                   \
-  static int LLVM_ATTRIBUTE_UNUSED X##ModuleAnchorDestination =                \
-      X##ModuleAnchorSource
-LINK_TIDY_MODULE(CERT);
-LINK_TIDY_MODULE(Abseil);
-LINK_TIDY_MODULE(Boost);
-LINK_TIDY_MODULE(Bugprone);
-LINK_TIDY_MODULE(LLVM);
-LINK_TIDY_MODULE(CppCoreGuidelines);
-LINK_TIDY_MODULE(Fuchsia);
-LINK_TIDY_MODULE(Google);
-LINK_TIDY_MODULE(Android);
-LINK_TIDY_MODULE(Misc);
-LINK_TIDY_MODULE(Modernize);
-LINK_TIDY_MODULE(Performance);
-LINK_TIDY_MODULE(Portability);
-LINK_TIDY_MODULE(Readability);
-LINK_TIDY_MODULE(ObjC);
-LINK_TIDY_MODULE(HICPP);
-LINK_TIDY_MODULE(Zircon);
-#undef LINK_TIDY_MODULE
-} // namespace tidy
 } // namespace clang
+
+#include "../clang-tidy/ClangTidyForceLinker.h"
\ No newline at end of file
Index: clangd/CMakeLists.txt
===================================================================
--- clangd/CMakeLists.txt
+++ clangd/CMakeLists.txt
@@ -107,6 +107,12 @@
   ${CLANGD_ATOMIC_LIB}
   )
 
+if( CLANG_ENABLE_STATIC_ANALYZER )
+  target_link_libraries(clangDaemon PRIVATE
+    clangTidyMPIModule
+  )
+endif()
+
 if( LLVM_LIB_FUZZING_ENGINE OR LLVM_USE_SANITIZE_COVERAGE )
   add_subdirectory(fuzzer)
 endif()

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits