[PATCH] D146784: [clang] Test for AST Dumping of the template variables

Tomasz KamiƄski via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Mar 24 06:51:46 PDT 2023


tomasz-kaminski-sonarsource added inline comments.


================
Comment at: clang/test/AST/ast-dump-template-decls.cpp:193
+// DIRECT:     VarTemplateSpecializationDecl 0x{{[^ ]*}} <line:{{[0-9]+}}:1, line:{{[0-9]+}}:16> col:5 unTempl 'int'
+// SERIALIZED: VarTemplateSpecializationDecl 0x{{[^ ]*}} <line:{{[0-9]+}}:1, line:{{[0-9]+}}:5> col:5 unTempl 'int'
+// CHECK-NEXT: `-TemplateArgument type 'int'
----------------
erichkeane wrote:
> Hmm... thats curious.  We shouldn't commit this unless there is a 'fixme' on it showing we don't really mean it, but in reality, this just deserves a Github bug.
Will add a FIXME. Feel free to create a GitHub bug.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D146784



More information about the cfe-commits mailing list