[PATCH] D128113: Clang: fix AST representation of expanded template arguments.

Matheus Izvekov via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat Aug 27 11:08:05 PDT 2022


mizvekov added a comment.

In D128113#3753662 <https://reviews.llvm.org/D128113#3753662>, @davrec wrote:

> Thanks for your work on this, and for explaining these complexities.

TBH I still don't know what to do here from the project perspective. I have a new deadline now that runs until November.

It seems that sitting down, figuring out a solution and coming up with a review-ready patch would take a considerable portion of that time. And then the review could take a year.

Meanwhile there is still considerable work remaining on the resugarer, in terms of addressing performance and getting everything into a reviewable state.

So I might try leaving this problem aside and not handle packs, for now. This would make us not able to resugar tuples for example, which is very unfortunate.

But in the unlikely case there was someone else interested in solving this problem, this could be parallelized.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D128113



More information about the cfe-commits mailing list