[PATCH] D52136: [clang-tidy] Add modernize-concat-nested-namespaces check

Andi via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 24 02:39:08 PST 2020


Abpostelnicu added inline comments.


================
Comment at: clang-tidy/modernize/ConcatNestedNamespacesCheck.cpp:88
+
+  if (!Sources.isInMainFile(ND.getBeginLoc()))
+    return;
----------------
This breaks the usage of this checker in an unified build scenario since the main file for the translation unit will be the unified file. 


Repository:
  rL LLVM

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

https://reviews.llvm.org/D52136





More information about the llvm-commits mailing list