From 848b7b389d6756d847d20373dfd7f0123fb7cc59 Mon Sep 17 00:00:00 2001 From: Argyrios Kyrtzidis Date: Thu, 6 Mar 2014 15:31:56 +0000 Subject: [PATCH] [ASTPrinter] 'SEL' isn't a ObjCObjectPointer, so we don't need to handle it in TypePrinter::printObjCObjectPointerBefore. Suggested by Jordan. git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@203124 91177308-0d34-0410-b5e6-96231b3b80d8 --- lib/AST/TypePrinter.cpp | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/lib/AST/TypePrinter.cpp b/lib/AST/TypePrinter.cpp index 9630ecd357..26c3c9903d 100644 --- a/lib/AST/TypePrinter.cpp +++ b/lib/AST/TypePrinter.cpp @@ -1281,12 +1281,12 @@ void TypePrinter::printObjCObjectPointerBefore(const ObjCObjectPointerType *T, T->getPointeeType().getLocalQualifiers().print(OS, Policy, /*appendSpaceIfNonEmpty=*/true); + assert(!T->isObjCSelType()); + if (T->isObjCIdType() || T->isObjCQualifiedIdType()) OS << "id"; else if (T->isObjCClassType() || T->isObjCQualifiedClassType()) OS << "Class"; - else if (T->isObjCSelType()) - OS << "SEL"; else OS << T->getInterfaceDecl()->getName(); @@ -1303,8 +1303,7 @@ void TypePrinter::printObjCObjectPointerBefore(const ObjCObjectPointerType *T, } if (!T->isObjCIdType() && !T->isObjCQualifiedIdType() && - !T->isObjCClassType() && !T->isObjCQualifiedClassType() && - !T->isObjCSelType()) { + !T->isObjCClassType() && !T->isObjCQualifiedClassType()) { OS << " *"; // Don't forget the implicit pointer. } else { spaceBeforePlaceHolder(OS); -- 2.40.0