diff options
| -rw-r--r-- | lib/CodeGen/TargetInfo.cpp | 11 | ||||
| -rw-r--r-- | test/CodeGen/arm-aapcs-vfp.c | 4 |
2 files changed, 8 insertions, 7 deletions
diff --git a/lib/CodeGen/TargetInfo.cpp b/lib/CodeGen/TargetInfo.cpp index 2cd239c81a..9c23ed9871 100644 --- a/lib/CodeGen/TargetInfo.cpp +++ b/lib/CodeGen/TargetInfo.cpp @@ -2757,13 +2757,10 @@ ABIArgInfo ARMABIInfo::classifyArgumentType(QualType Ty) const { } } - // Turn on byval for APCS and AAPCS. - // FIXME: turn on byval for AAPCS_VFP for performance. - if (getABIKind() == ARMABIInfo::APCS || getABIKind() == ARMABIInfo::AAPCS) { - if (getContext().getTypeSizeInChars(Ty) > CharUnits::fromQuantity(64) || - getContext().getTypeAlign(Ty) > 64) { - return ABIArgInfo::getIndirect(0, /*ByVal=*/true); - } + // Support byval for ARM. + if (getContext().getTypeSizeInChars(Ty) > CharUnits::fromQuantity(64) || + getContext().getTypeAlign(Ty) > 64) { + return ABIArgInfo::getIndirect(0, /*ByVal=*/true); } // Otherwise, pass by coercing to a structure of the appropriate size. diff --git a/test/CodeGen/arm-aapcs-vfp.c b/test/CodeGen/arm-aapcs-vfp.c index fdd87e0549..614b52dad5 100644 --- a/test/CodeGen/arm-aapcs-vfp.c +++ b/test/CodeGen/arm-aapcs-vfp.c @@ -88,3 +88,7 @@ extern void neon_callee(struct neon_struct); void test_neon(struct neon_struct arg) { neon_callee(arg); } + +// CHECK: define arm_aapcs_vfpcc void @f33(%struct.s33* byval %s) +struct s33 { char buf[32*32]; }; +void f33(struct s33 s) { } |