break;
case DeclaratorChunk::Array: {
const DeclaratorChunk::ArrayTypeInfo &ATI = DeclType.Arr;
+ Expr *ArraySize = static_cast<Expr*>(ATI.NumElts);
ArrayType::ArraySizeModifier ASM;
if (ATI.isStar)
ASM = ArrayType::Star;
D.setInvalidType(true);
}
}
- T = Context.getArrayType(T, ASM, ATI.TypeQuals,
- static_cast<Expr *>(ATI.NumElts));
+ T = Context.getArrayType(T, ASM, ATI.TypeQuals, ArraySize);
+
+ // If this is not C99, extwarn about VLA's and C99 array size modifiers.
+ if (!getLangOptions().C99 &&
+ (ASM != ArrayType::Normal ||
+ (ArraySize && !ArraySize->isIntegerConstantExpr(Context))))
+ Diag(D.getIdentifierLoc(), diag::ext_vla);
break;
}
case DeclaratorChunk::Function:
"ISO C requires a named argument before '...'")
DIAG(err_unspecified_vla_size_with_static, ERROR,
"'static' may not be used with an unspecified variable length array size")
+DIAG(ext_vla, EXTENSION,
+ "variable length arrays are a C99 feature, accepted as an extension")
DIAG(err_invalid_storage_class_in_func_decl, ERROR,
"invalid storage class specifier in function declarator")
DIAG(err_invalid_reference_qualifier_application, ERROR,
/* RUN: clang %s -std=c89 -pedantic -parse-ast-check
*/
-void foo() {
+void test1() {
{
int i;
i = i + 1;
}
}
-long long x; /* expected-warning {{extension}} */
+long long test2; /* expected-warning {{extension}} */
+
+void test3(int i) {
+ int A[i]; /* expected-warning {{variable length array}} */
+}