From: Chris Lattner Date: Sat, 4 Sep 2010 23:16:01 +0000 (+0000) Subject: revise r112365 to fix the actual problem: the isa(Underlying) X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=c3f8c0731ef59ba79753f89f1c108b8134f6ae83;p=clang revise r112365 to fix the actual problem: the isa(Underlying) check in the "typedef for anonymous type" check should have been a getAs. git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@113085 91177308-0d34-0410-b5e6-96231b3b80d8 --- diff --git a/lib/AST/ASTDiagnostic.cpp b/lib/AST/ASTDiagnostic.cpp index 23f323d028..321a139665 100644 --- a/lib/AST/ASTDiagnostic.cpp +++ b/lib/AST/ASTDiagnostic.cpp @@ -81,10 +81,10 @@ break; \ break; // Don't desugar through the primary typedef of an anonymous type. - if (isa(Underlying) && isa(QT)) - if (cast(Underlying)->getDecl()->getTypedefForAnonDecl() == - cast(QT)->getDecl()) - break; + if (const TagType *UTT = Underlying->getAs()) + if (const TypedefType *QTT = dyn_cast(QT)) + if (UTT->getDecl()->getTypedefForAnonDecl() == QTT->getDecl()) + break; // Record that we actually looked through an opaque type here. ShouldAKA = true; @@ -94,11 +94,11 @@ break; \ // If we have a pointer-like type, desugar the pointee as well. // FIXME: Handle other pointer-like types. if (const PointerType *Ty = QT->getAs()) { - QT = Context.getPointerType(Desugar(Context, Ty->getPointeeType(), - ShouldAKA)); + QT = Context.getPointerType(Desugar(Context, Ty->getPointeeType(), + ShouldAKA)); } else if (const LValueReferenceType *Ty = QT->getAs()) { - QT = Context.getLValueReferenceType(Desugar(Context, Ty->getPointeeType(), - ShouldAKA)); + QT = Context.getLValueReferenceType(Desugar(Context, Ty->getPointeeType(), + ShouldAKA)); } return QC.apply(QT); @@ -151,13 +151,10 @@ ConvertTypeToDiagnosticString(ASTContext &Context, QualType Ty, bool ShouldAKA = false; QualType DesugaredTy = Desugar(Context, Ty, ShouldAKA); if (ShouldAKA) { - std::string D = DesugaredTy.getAsString(Context.PrintingPolicy); - if (D != S) { - S = "'" + S + "' (aka '"; - S += D; - S += "')"; - return S; - } + S = "'" + S + "' (aka '"; + S += DesugaredTy.getAsString(Context.PrintingPolicy); + S += "')"; + return S; } }