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

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

Reply via email to