Fix tests

Author: YexuanXiao

clang-tools-extra/clangd/unittests/HoverTests.cpp

@@ -2794,7 +2794,7 @@ TEST(Hover, All) {
           })cpp",
           [](HoverInfo &HI) {
             HI.Name = "expression";
-            HI.Type = "__size_t (aka unsigned long)";
+            HI.Type = {"__size_t", "unsigned long"};
             HI.Value = "1";
           }},
       {
@@ -2804,7 +2804,7 @@ TEST(Hover, All) {
           })cpp",
           [](HoverInfo &HI) {
             HI.Name = "expression";
-            HI.Type = "__size_t (aka unsigned long)";
+            HI.Type = {"__size_t", "unsigned long"};
             HI.Value = "1";
           }},
       {

clang/lib/AST/ASTContext.cpp

@@ -14598,7 +14598,9 @@ static QualType getCommonSugarTypeNode(ASTContext &Ctx, const Type *X,
                                       DX->isCountInBytes(), DX->isOrNull(),
                                       CDX);
   }
-  case Type::PredefinedSugar:;
+  case Type::PredefinedSugar:
+    // FIXME: Should this be reachable here?
+    return QualType();
   }
   llvm_unreachable("Unhandled Type Class");
 }

clang/lib/CodeGen/CodeGenFunction.cpp

@@ -722,7 +722,7 @@ static bool matchesStlAllocatorFn(const Decl *D, const ASTContext &Ctx) {
     return false;
 
   if (MD->parameters()[0]->getType().getCanonicalType() !=
-    Ctx.getCanonicalSizeType())
+      Ctx.getCanonicalSizeType())
     return false;
 
   if (MD->getNumParams() == 2) {

clang/lib/Sema/TreeTransform.h

@@ -7257,7 +7257,8 @@ QualType TreeTransform<Derived>::TransformPredefinedSugarType(
   QualType Result = TL.getType();
 
   if (getDerived().AlwaysRebuild()) {
-    Result = getDerived().RebuildPredefinedSugarType(llvm::to_underlying(EIT->getKind()));
+    Result = getDerived().RebuildPredefinedSugarType(
+        llvm::to_underlying(EIT->getKind()));
   }
 
   PredefinedSugarTypeLoc NewTL = TLB.push<PredefinedSugarTypeLoc>(Result);