diff options
| author | Oliver Stannard <oliver.stannard@arm.com> | 2014-06-27 13:59:27 +0000 |
|---|---|---|
| committer | Oliver Stannard <oliver.stannard@arm.com> | 2014-06-27 13:59:27 +0000 |
| commit | dc111a9e363412c7b1a3387333cb81049678c738 (patch) | |
| tree | eeb3fa54607fae7cc0d3bcadbfca47f1c9574fca /test | |
| parent | 69f87edbebe7dc9b450482a4af0a3d6b55ef5c23 (diff) | |
| download | clang-dc111a9e363412c7b1a3387333cb81049678c738.tar.gz clang-dc111a9e363412c7b1a3387333cb81049678c738.tar.bz2 clang-dc111a9e363412c7b1a3387333cb81049678c738.tar.xz | |
[ARM] Fix AAPCS non-compliance caused by very large structs
This is a fix to the code in clang which inserts padding arguments to
ensure that the ARM backend can emit AAPCS-VFP compliant code. This code
needs to track the number of registers which have been allocated in order
to do this. When passing a very large struct (>64 bytes) by value, clang
emits IR which takes a pointer to the struct, but the backend converts this
back to passing the struct in registers and on the stack. The bug was that
this was being considered by clang to only use one register, meaning that
there were situations in which padding arguments were incorrectly emitted
by clang.
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@211898 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'test')
| -rw-r--r-- | test/CodeGen/arm-aapcs-vfp.c | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/test/CodeGen/arm-aapcs-vfp.c b/test/CodeGen/arm-aapcs-vfp.c index 96fd625701..7bc1b1e55c 100644 --- a/test/CodeGen/arm-aapcs-vfp.c +++ b/test/CodeGen/arm-aapcs-vfp.c @@ -139,3 +139,9 @@ void test_test_vfp_stack_gpr_split_bitfield(double a, double b, double c, double typedef struct { int x; long long y; } struct_int_long_long; // CHECK: define arm_aapcs_vfpcc void @test_vfp_stack_gpr_split_4(double %a, double %b, double %c, double %d, double %e, double %f, double %g, double %h, double %i, i32 %j, [3 x i32], { [2 x i64] } %k.coerce) void test_vfp_stack_gpr_split_4(double a, double b, double c, double d, double e, double f, double g, double h, double i, int j, struct_int_long_long k) {} + +// This very large struct (passed byval) uses up the GPRs, so no padding is needed +typedef struct { int x[17]; } struct_seventeen_ints; +typedef struct { int x[4]; } struct_four_ints; +// CHECK: define arm_aapcs_vfpcc void @test_vfp_stack_gpr_split_5(%struct.struct_seventeen_ints* byval align 4 %a, double %b, double %c, double %d, double %e, double %f, double %g, double %h, double %i, double %j, { [4 x i32] } %k.coerce) +void test_vfp_stack_gpr_split_5(struct_seventeen_ints a, double b, double c, double d, double e, double f, double g, double h, double i, double j, struct_four_ints k) {} |