HerrCai0907 added inline comments.
================ Comment at: clang/lib/Sema/SemaType.cpp:2582 } else if (ArraySize->isTypeDependent() || ArraySize->isValueDependent()) { - T = Context.getDependentSizedArrayType(T, ArraySize, ASM, Quals, Brackets); + if (getLangOpts().CPlusPlus) { + T = Context.getDependentSizedArrayType(T, ArraySize, ASM, Quals, ---------------- erichkeane wrote: > So I'm still not sure this is the 'right' away about it. I think we should > instead properly handle the `containsErrors` case and just always create a > non-dependent sized array, except with the `RecoveryExpr` having the correct > type. For CPP, dependent sized array is acceptable and necessary. For C, I don't think SemaType is a correctly way to resolve TypoExpr, It should be done by `ActOnXXX`. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D149612/new/ https://reviews.llvm.org/D149612 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits