mizvekov added a comment.

In D133262#3768883 <https://reviews.llvm.org/D133262#3768883>, @royjacobson 
wrote:

> There are some known bugs about how we handle this built-in in the AST- 
> #42102, #51928, #54993. Is it possible that your patch solves them? It would 
> be great if it does, I hit one of them in the wild a while back.

I think not, these look like deduction problems, and what is done in this patch 
should not affect deduction.

The TemplateSpecializationTypes we are changing in this patch are all sugar, 
and these should not affect it.

Dependence can create canonical TST, which does affect it, but 
`checkBuiltinTemplateIdType` is not even called if there is any dependence.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D133262

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

Reply via email to