fix a builder, why didn't clang++ catch this?

git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@109735 91177308-0d34-0410-b5e6-96231b3b80d8

diff --git a/lib/CodeGen/ABIInfo.h b/lib/CodeGen/ABIInfo.h
index 642fddb5c8dd8651bd94a036dda4b25cee1942fb..ade650580c384f6c50d28b02764b08449e112167 100644 (file)
--- a/lib/CodeGen/ABIInfo.h
+++ b/lib/CodeGen/ABIInfo.h
@@ -52,7 +52,7 @@ namespace clang {
       Ignore,    /// Ignore the argument (treat as void). Useful for
                  /// void and empty structs.
 
-       Expand,    /// Only valid for aggregate argument types. The
+      Expand,    /// Only valid for aggregate argument types. The
                  /// structure should be expanded into consecutive
                  /// arguments for its constituent fields. Currently
                  /// expand is only allowed on structures whose fields

diff --git a/lib/CodeGen/CGCall.cpp b/lib/CodeGen/CGCall.cpp
index 94cfd98d07d81d13b401f61d8459db2f3c848ab2..94a2ae9ee65943bbd0347b1055842be327fc90e5 100644 (file)
--- a/lib/CodeGen/CGCall.cpp
+++ b/lib/CodeGen/CGCall.cpp
@@ -636,7 +636,7 @@ CodeGenTypes::GetFunctionType(const CGFunctionInfo &FI, bool IsVariadic,
     }
 
     case ABIArgInfo::Extend:
-    case ABIArgInfo::Direct:
+    case ABIArgInfo::Direct: {
       // If the coerce-to type is a first class aggregate, flatten it.  Either
       // way is semantically identical, but fast-isel and the optimizer
       // generally likes scalar values better than FCAs.
@@ -648,6 +648,7 @@ CodeGenTypes::GetFunctionType(const CGFunctionInfo &FI, bool IsVariadic,
         ArgTys.push_back(ArgTy);
       }
       break;
+    }
 
     case ABIArgInfo::Expand:
       GetExpandedTypes(it->type, ArgTys, IsRecursive);