From 1ed72678b41c6038b08f48cb7e2a7b61e2dd9179 Mon Sep 17 00:00:00 2001 From: Chris Lattner Date: Thu, 29 Jul 2010 06:44:09 +0000 Subject: [PATCH] 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 --- lib/CodeGen/ABIInfo.h | 2 +- lib/CodeGen/CGCall.cpp | 3 ++- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/lib/CodeGen/ABIInfo.h b/lib/CodeGen/ABIInfo.h index 642fddb5c8..ade650580c 100644 --- 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 94cfd98d07..94a2ae9ee6 100644 --- 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); -- 2.40.0