erichkeane added a comment.

In D126907#3565491 <https://reviews.llvm.org/D126907#3565491>, @ChuanqiXu wrote:

> Oh, the diff part is small really. The change looks good to me and I've 
> tested it for libcxx.
>
> But it crashed at "clang/lib/Sema/SemaTemplateInstantiate.cpp:1852: 
> clang::QualType 
> {anonymous}::TemplateInstantiator::TransformTemplateTypeParmType(clang::TypeLocBuilder&,
>  clang::TemplateTypeParmTypeLoc): Assertion `Arg.getKind() == 
> TemplateArgument::Type && "Template argument kind mismatch"' failed" in an 
> internal library. I am trying to get a reduced example. I would suggest to do 
> more testing with other workloads (with libstdc++ >= 10)

Thanks so much for the feedback, and for running it!   I also checked out the 
basics of libcxx, and it worked fine (as well as SOME libstdc++ repros, but not 
extensively).  Any reproducer you could give me would be great!


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

https://reviews.llvm.org/D126907

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

Reply via email to