Index: clang/lib/Sema/SemaType.cpp =================================================================== --- clang/lib/Sema/SemaType.cpp +++ clang/lib/Sema/SemaType.cpp @@ -2175,19 +2175,21 @@ T = Context.getConstantArrayType(T, ConstVal, ASM, Quals); } - // OpenCL v1.2 s6.9.d: variable length arrays are not supported. - if (getLangOpts().OpenCL && T->isVariableArrayType()) { - Diag(Loc, diag::err_opencl_vla); - return QualType(); - } - // CUDA device code doesn't support VLAs. - if (getLangOpts().CUDA && T->isVariableArrayType()) - CUDADiagIfDeviceCode(Loc, diag::err_cuda_vla) << CurrentCUDATarget(); - // Some targets don't support VLAs. - if (T->isVariableArrayType() && !Context.getTargetInfo().isVLASupported() && - shouldDiagnoseTargetSupportFromOpenMP()) { - Diag(Loc, diag::err_vla_unsupported); - return QualType(); + if (T->isVariableArrayType()) { + if (getLangOpts().OpenCL) { + // OpenCL v1.2 s6.9.d: variable length arrays are not supported. + Diag(Loc, diag::err_opencl_vla); + return QualType(); + } else if (getLangOpts().CUDA) { + // CUDA device code doesn't support VLAs. + CUDADiagIfDeviceCode(Loc, diag::err_cuda_vla) << CurrentCUDATarget(); + } else if (getLangOpts().OpenMP && + !Context.getTargetInfo().isVLASupported() && + shouldDiagnoseTargetSupportFromOpenMP()) { + // Some targets don't support VLAs. + Diag(Loc, diag::err_vla_unsupported); + return QualType(); + } } // If this is not C99, extwarn about VLA's and C99 array size modifiers.