[PATCH] D55393: Re-order content of template parameter dumps

Aaron Ballman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Dec 7 05:37:23 PST 2018


aaron.ballman added inline comments.


================
Comment at: test/AST/ast-dump-decl.cpp:330-331
 // CHECK-NEXT:   TemplateTypeParmDecl
-// CHECK-NEXT:     TemplateArgument type 'int'
 // CHECK-NEXT:     inherited from TemplateTypeParm 0x{{[^ ]*}} 'T'
+// CHECK-NEXT:     TemplateArgument type 'int'
 
----------------
steveire wrote:
> aaron.ballman wrote:
> > This looks wrong to me. You did not inherit the `TemplateTypeParmDecl` from above, you inherited the `TemplateArgument`. The order is important here and should be preserved to avoid confusion.
> Should this be dumped as a child of the `TemplateArgument` then? I can add it as a child in `ASTDumper::dumpTemplateArgument`.
I kind of think it should. We follow the same pattern in each place -- print the template argument, then print this information, so combining it seems reasonable to me.


Repository:
  rC Clang

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

https://reviews.llvm.org/D55393





More information about the cfe-commits mailing list