diff options
author | Eli Friedman <eli.friedman@gmail.com> | 2012-11-29 03:13:49 +0000 |
---|---|---|
committer | Eli Friedman <eli.friedman@gmail.com> | 2012-11-29 03:13:49 +0000 |
commit | fd41aee2f85891396ba7e0df939a2b96301bed80 (patch) | |
tree | 2742dc8c278e8b44caedda2cc970c3b5172c555d /clang/lib/Sema/SemaPseudoObject.cpp | |
parent | 7afe1663e9de64118694d2399ffa2743da1980a7 (diff) | |
download | bcm5719-llvm-fd41aee2f85891396ba7e0df939a2b96301bed80.tar.gz bcm5719-llvm-fd41aee2f85891396ba7e0df939a2b96301bed80.zip |
Fix crash-on-invalid. <rdar://problem/12765391>.
llvm-svn: 168851
Diffstat (limited to 'clang/lib/Sema/SemaPseudoObject.cpp')
-rw-r--r-- | clang/lib/Sema/SemaPseudoObject.cpp | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/clang/lib/Sema/SemaPseudoObject.cpp b/clang/lib/Sema/SemaPseudoObject.cpp index 74d0818b0e4..67bf7423844 100644 --- a/clang/lib/Sema/SemaPseudoObject.cpp +++ b/clang/lib/Sema/SemaPseudoObject.cpp @@ -113,7 +113,7 @@ namespace { Expr *rebuildSpecific(ObjCPropertyRefExpr *refExpr) { // Fortunately, the constraint that we're rebuilding something // with a base limits the number of cases here. - assert(refExpr->getBase()); + assert(refExpr->isObjectReceiver()); if (refExpr->isExplicitProperty()) { return new (S.Context) @@ -713,10 +713,9 @@ ExprResult ObjCPropertyOpBuilder::buildSet(Expr *op, SourceLocation opcLoc, ExprResult ObjCPropertyOpBuilder::buildRValueOperation(Expr *op) { // Explicit properties always have getters, but implicit ones don't. // Check that before proceeding. - if (RefExpr->isImplicitProperty() && - !RefExpr->getImplicitPropertyGetter()) { + if (RefExpr->isImplicitProperty() && !RefExpr->getImplicitPropertyGetter()) { S.Diag(RefExpr->getLocation(), diag::err_getter_not_found) - << RefExpr->getBase()->getType(); + << RefExpr->getSourceRange(); return ExprError(); } |