[PATCH] D133262: [clang] Represent __make_integer_seq as alias template in the AST

Matheus Izvekov via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat Sep 3 16:39:29 PDT 2022


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



More information about the cfe-commits mailing list