carlosgalvezp added inline comments.

================
Comment at: clang-tools-extra/clang-tidy/llvmlibc/LLVMLibcTidyModule.cpp:28
+    CheckFactories.registerCheck<InlineFunctionDeclCheck>(
+        "llvmlibc-inline-function-decl-check");
     CheckFactories.registerCheck<RestrictSystemLibcHeadersCheck>(
----------------
Remove


================
Comment at: clang-tools-extra/docs/ReleaseNotes.rst:100
 
+New `llvmlibc-inline-function-decl-check 
<http://clang.llvm.org/extra/clang-tidy/checks/llvmlibc/inline-function-decl-check.html>`_
+check.
----------------
carlosgalvezp wrote:
> I believe you'll need a rebase since a new check was recently added.
Remove


================
Comment at: clang-tools-extra/docs/ReleaseNotes.rst:100
 
+New `llvmlibc-inline-function-decl-check 
<http://clang.llvm.org/extra/clang-tidy/checks/llvmlibc/inline-function-decl-check.html>`_
+check.
----------------
carlosgalvezp wrote:
> carlosgalvezp wrote:
> > I believe you'll need a rebase since a new check was recently added.
> Remove
Remove


================
Comment at: 
clang-tools-extra/docs/clang-tidy/checks/llvmlibc/inline-function-decl-check.rst:1
+.. title:: clang-tidy - llvmlibc-inline-funciton-decl-check
+
----------------
Remove


================
Comment at: 
clang-tools-extra/docs/clang-tidy/checks/llvmlibc/inline-function-decl-check.rst:3
+
+llvmlibc-inline-function-decl-check
+====================================
----------------
Remove


================
Comment at: 
clang-tools-extra/docs/clang-tidy/checks/llvmlibc/inline-function-decl-check.rst:4
+llvmlibc-inline-function-decl-check
+====================================
+
----------------
Nit: remove extra `=`


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D142592

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

Reply via email to