[SemaCXX]use CorrectDelayedTyposInExpr in ActOnCXXFoldExpr only when Diag
authorCongcong Cai <congcongcai0907@163.com>
Tue, 14 Mar 2023 17:07:55 +0000 (01:07 +0800)
committerCongcong Cai <congcongcai0907@163.com>
Tue, 14 Mar 2023 17:08:41 +0000 (01:08 +0800)
commite417f02b5159c13f011335636faaf8c6847b627f
treec3d57acd74bc359544323a83543290efd13ad9e8
parent507cba21c382ced5fc075ac74637847fbc2a4c07
[SemaCXX]use CorrectDelayedTyposInExpr in ActOnCXXFoldExpr only when Diag

PR #61326

- fix clang crash when fold expression contains a delayed typos correction.

code snippet in `ActOnCXXFoldExpr`
```  if (!LHS || !RHS) {
    Expr *Pack = LHS ? LHS : RHS;
    assert(Pack && "fold expression with neither LHS nor RHS");
    DiscardOperands();
    if (!Pack->containsUnexpandedParameterPack())
      return Diag(EllipsisLoc, diag::err_pack_expansion_without_parameter_packs)
             << Pack->getSourceRange();
  }
```
`DiscardOperands` will be triggered when LHS/RHS is delayed typo correction expression.
It will output and clean all diagnose but still return a valid expression. (in else branch)
valid expression will be handled in caller function. When caller wants to output the diagnose, the diagnose in delayed typo correction expression has been consumed in `ActOnCXXFoldExpr`. It causes clang crash.

Reviewed By: erichkeane

Differential Revision: https://reviews.llvm.org/D145892
clang/docs/ReleaseNotes.rst
clang/lib/Sema/SemaTemplateVariadic.cpp
clang/test/SemaCXX/fold_expr_typo.cpp [new file with mode: 0644]