[PATCH] D156693: [clang][ASTImporter]Skip check depth of friend template parameter

Qizhi Hu via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Aug 21 18:46:23 PDT 2023


jcsxky added inline comments.


================
Comment at: clang/unittests/AST/StructuralEquivalenceTest.cpp:140
+        D0->getASTContext(), D1->getASTContext(), NonEquivalentDecls01,
+        StructuralEquivalenceKind::Default, false, false, false,
+        IgnoreTemplateParmDepth);
----------------
shafik wrote:
> See [bugprone-argument-comment](https://clang.llvm.org/extra/clang-tidy/checks/bugprone/argument-comment.html) for details of this format. 
> 
> We use this everywhere we have literals without clear meaning being used as arguments. 
argument comment has been added.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D156693



More information about the cfe-commits mailing list