From: Benjamin Kramer <benny.kra@googlemail.com>
Date: Sat, 5 Dec 2009 22:16:51 +0000 (+0000)
Subject: Don't call back() on an empty vector.
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=b2041de2eb3e44d04cb8c62c4369314af4fd6a90;p=clang

Don't call back() on an empty vector.


git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@90678 91177308-0d34-0410-b5e6-96231b3b80d8
---

diff --git a/lib/Frontend/RewriteObjC.cpp b/lib/Frontend/RewriteObjC.cpp
index fc5ec2e9d0..08b54b2222 100644
--- a/lib/Frontend/RewriteObjC.cpp
+++ b/lib/Frontend/RewriteObjC.cpp
@@ -4591,8 +4591,7 @@ Stmt *RewriteObjC::RewriteFunctionBodyOrGlobalInitializer(Stmt *S) {
     //   for (id <FooProtocol> index in someArray) ;
     // This is because RewriteObjCForCollectionStmt() does textual rewriting 
     // and it depends on the original text locations/positions.
-    Stmt *ParentStmt = Stmts.back();
-    if (!ParentStmt || !isa<ObjCForCollectionStmt>(ParentStmt))
+    if (Stmts.empty() || !isa<ObjCForCollectionStmt>(Stmts.back()))
       RewriteObjCQualifiedInterfaceTypes(*DS->decl_begin());
 
     // Blocks rewrite rules.