diff options
| -rw-r--r-- | clang/lib/Sema/SemaExpr.cpp | 6 | ||||
| -rw-r--r-- | clang/test/Sema/builtins.cl | 5 |
2 files changed, 9 insertions, 2 deletions
diff --git a/clang/lib/Sema/SemaExpr.cpp b/clang/lib/Sema/SemaExpr.cpp index ac5da811299..0abe75ab2d8 100644 --- a/clang/lib/Sema/SemaExpr.cpp +++ b/clang/lib/Sema/SemaExpr.cpp @@ -5051,7 +5051,11 @@ static FunctionDecl *rewriteBuiltinFunctionDecl(Sema *Sema, ASTContext &Context, for (QualType ParamType : FT->param_types()) { // Convert array arguments to pointer to simplify type lookup. - Expr *Arg = Sema->DefaultFunctionArrayLvalueConversion(ArgExprs[i++]).get(); + ExprResult ArgRes = + Sema->DefaultFunctionArrayLvalueConversion(ArgExprs[i++]); + if (ArgRes.isInvalid()) + return nullptr; + Expr *Arg = ArgRes.get(); QualType ArgType = Arg->getType(); if (!ParamType->isPointerType() || ParamType.getQualifiers().hasAddressSpace() || diff --git a/clang/test/Sema/builtins.cl b/clang/test/Sema/builtins.cl index 8cde8f3d10f..7cde5e1a9e4 100644 --- a/clang/test/Sema/builtins.cl +++ b/clang/test/Sema/builtins.cl @@ -1,8 +1,11 @@ // RUN: %clang_cc1 %s -fsyntax-only -verify -pedantic -// expected-no-diagnostics kernel void test(global float *out, global float *in, global int* in2) { out[0] = __builtin_nanf(""); __builtin_memcpy(out, in, 32); out[0] = __builtin_frexpf(in[0], in2); } + +void pr28651() { + __builtin_alloca(value); // expected-error{{use of undeclared identifier}} +} |

