aaron.ballman added inline comments.

================
Comment at: clang-tidy/fuchsia/CMakeLists.txt:6
   FuchsiaTidyModule.cpp
+  ThreadLocalCheck.cpp
   OverloadedOperatorCheck.cpp
----------------
Please keep this list in alphabetical order.


================
Comment at: clang-tidy/fuchsia/ThreadLocalCheck.cpp:27
+  if (const auto *D = Result.Nodes.getNodeAs<VarDecl>("decl"))
+    diag(D->getLocStart(), "thread local storage is disallowed");
+}
----------------
The docs say it's only disallowed in kernel mode. Can you remind me again -- 
are the fuchsia checks *only* for kernel mode, or is that the "zircon" part? 
I'm wondering if this check is overly broad for the guidelines, basically.


https://reviews.llvm.org/D41963



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

Reply via email to