================
@@ -831,7 +831,7 @@ class PackDeductionScope {
if (IsPartiallyExpanded)
PackElements += NumPartialPackArgs;
else if (IsExpanded)
- PackElements += *FixedNumExpansions;
+ PackElements += FixedNumExpansions.value_or(1);
----------------
mizvekov wrote:
The assert is not necessary, unless it added explanation.
The `*` operator on the optional here should already assert in that case.
I would advise you run this test case on an llvm build with runtime checks
enabled. The compiler is going off the rails before this point, so maybe there
is an earlier assert that could narrow it down.
Otherwise, the change lacks explanation, about what is the problem, and what
the fix does.
https://github.com/llvm/llvm-project/pull/88637
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
