NFC: Canonicalize handling of TypeLocInfo

[steveire]

Summary

No need to avoid the Visit method.

[aaron.ballman]

LGTM!

lib/AST/ASTDumper.cpp

public:
 
     // Objective-C utilities.
     void dumpObjCTypeParamList(const ObjCTypeParamList *typeParams);
 
     // Types
     void VisitComplexType(const ComplexType *T) {
       dumpTypeAsChild(T->getElementType());
     }
+    void VisitLocInfoType(const LocInfoType *T) {
+      dumpTypeAsChild(T->getTypeSourceInfo()->getType());
+    }
     void VisitPointerType(const PointerType *T) {
       dumpTypeAsChild(T->getPointeeType());
     }
     void VisitBlockPointerType(const BlockPointerType *T) {
       dumpTypeAsChild(T->getPointeeType());
     }
     void VisitReferenceType(const ReferenceType *T) {
       dumpTypeAsChild(T->getPointeeType());

 }
 
 void ASTDumper::dumpTypeAsChild(const Type *T) {
   dumpChild([=] {
     NodeDumper.Visit(T);
     if (!T) {
       return;
     }
-    if (const LocInfoType *LIT = llvm::dyn_cast<LocInfoType>(T)) {
-      dumpTypeAsChild(LIT->getTypeSourceInfo()->getType());
-      return;
-    }
     TypeVisitor<ASTDumper>::Visit(T);
 
     QualType SingleStepDesugar =
         T->getLocallyUnqualifiedSingleStepDesugaredType();
     if (SingleStepDesugar != QualType(T, 0))
       dumpTypeAsChild(SingleStepDesugar);
   });
 }