[PATCH] D23130: [Clang-tidy] Add a check for definitions in the global namespace.

Malcolm Parsons via cfe-commits cfe-commits at lists.llvm.org
Thu Nov 10 02:02:22 PST 2016


malcolm.parsons added inline comments.


================
Comment at: clang-tidy/google/GlobalNamesCheck.cpp:96
+    // main() should be in the global namespace.
+    if (FDecl->isMain())
+      return;
----------------
Should `isMSVCRTEntryPoint()` be checked too?


https://reviews.llvm.org/D23130





More information about the cfe-commits mailing list