[clang] Fix compilation error in pp-trace and remove IdentifierInfo forward declaration in SourceLocation

Signed-off-by: yronglin <[email protected]>

Author: yronglin

clang-tools-extra/pp-trace/PPCallbacksTracker.cpp

@@ -547,8 +547,8 @@ void PPCallbacksTracker::appendArgument(const char *Name, ModuleIdPath Value) {
     if (I)
       SS << ", ";
     SS << "{"
-       << "Name: " << Value[I].first->getName() << ", "
-       << "Loc: " << getSourceLocationString(PP, Value[I].second) << "}";
+       << "Name: " << Value[I].getIdentifierInfo()->getName() << ", "
+       << "Loc: " << getSourceLocationString(PP, Value[I].getLoc()) << "}";
   }
   SS << "]";
   appendArgument(Name, SS.str());

clang/include/clang/Basic/SourceLocation.h

@@ -31,7 +31,6 @@ template <typename T, typename Enable> struct FoldingSetTrait;
 
 namespace clang {
 
-class IdentifierInfo;
 class SourceManager;
 
 /// An opaque identifier used by SourceManager which refers to a