[PATCH] D139837: [Clang] Implements CTAD for aggregates P1816R0 and P2082R1

[Corentin Jabot via Phabricator via cfe-commits]

cor3ntin added inline comments.


================
Comment at: clang/lib/Sema/SemaTemplateInstantiate.cpp:1059-1062
+    case CodeSynthesisContext::BuildingDeductionGuides:
+      assert(
+          false &&
+          "failed building deduction guides, add meaningful diagnostics here");
----------------
ychen wrote:
> aaron.ballman wrote:
> > cor3ntin wrote:
> > > This seems unfinished
> > +1
> I meant to keep this a future work since this path is dead until some errors could be thrown out of this context.  In the future, if errors happen during building deduction guides, this assertion failure could trigger at build time.
In this case, maybe change the message
assert(false && "unexpected deduction guide in instantiation stack") or something along those lines.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D139837