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

llvm-svn: 109735

2 files changed, 3 insertions, 2 deletions

diff --git a/clang/lib/CodeGen/ABIInfo.h b/clang/lib/CodeGen/ABIInfo.h
index 642fddb5c8d..ade650580c3 100644
--- a/clang/lib/CodeGen/ABIInfo.h
+++ b/clang/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/clang/lib/CodeGen/CGCall.cpp b/clang/lib/CodeGen/CGCall.cpp
index 94cfd98d07d..94a2ae9ee65 100644
--- a/clang/lib/CodeGen/CGCall.cpp
+++ b/clang/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);