diff options
| author | Chris Lattner <sabre@nondot.org> | 2009-04-25 22:50:55 +0000 |
|---|---|---|
| committer | Chris Lattner <sabre@nondot.org> | 2009-04-25 22:50:55 +0000 |
| commit | 003af249275a767b6bb6a56b8b4af2fb6596bdd0 (patch) | |
| tree | aa298f0c2212202012745c7c9af648668895b58b /clang/lib | |
| parent | 50322773b787882f2510f141922755ff267d6145 (diff) | |
| download | bcm5719-llvm-003af249275a767b6bb6a56b8b4af2fb6596bdd0.tar.gz bcm5719-llvm-003af249275a767b6bb6a56b8b4af2fb6596bdd0.zip | |
minor diagnostics improvements.
llvm-svn: 70092
Diffstat (limited to 'clang/lib')
| -rw-r--r-- | clang/lib/Sema/SemaExpr.cpp | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/clang/lib/Sema/SemaExpr.cpp b/clang/lib/Sema/SemaExpr.cpp index 6e86e71bd4a..873dd53f12c 100644 --- a/clang/lib/Sema/SemaExpr.cpp +++ b/clang/lib/Sema/SemaExpr.cpp @@ -1636,13 +1636,13 @@ Sema::ActOnArraySubscriptExpr(Scope *S, ExprArg Base, SourceLocation LLoc, // FIXME: need to deal with const... ResultType = VTy->getElementType(); } else { - return ExprError(Diag(LHSExp->getLocStart(), - diag::err_typecheck_subscript_value) << RHSExp->getSourceRange()); + return ExprError(Diag(LLoc, diag::err_typecheck_subscript_value) + << LHSExp->getSourceRange() << RHSExp->getSourceRange()); } // C99 6.5.2.1p1 if (!IndexExpr->getType()->isIntegerType() && !IndexExpr->isTypeDependent()) - return ExprError(Diag(IndexExpr->getLocStart(), - diag::err_typecheck_subscript) << IndexExpr->getSourceRange()); + return ExprError(Diag(LLoc, diag::err_typecheck_subscript_not_integer) + << IndexExpr->getSourceRange()); // C99 6.5.2.1p1: "shall have type "pointer to *object* type". Similarly, // C++ [expr.sub]p1: The type "T" shall be a completely-defined object @@ -4586,7 +4586,7 @@ Sema::OwningExprResult Sema::ActOnBuiltinOffsetOf(Scope *S, // FIXME: Leaks Res if (!Idx->isTypeDependent() && !Idx->getType()->isIntegerType()) return ExprError(Diag(Idx->getLocStart(), - diag::err_typecheck_subscript) + diag::err_typecheck_subscript_not_integer) << Idx->getSourceRange()); Res = new (Context) ArraySubscriptExpr(Res, Idx, AT->getElementType(), |
