diff options
| author | Richard Sandiford <richard.sandiford@arm.com> | 2019-11-08 09:06:50 +0000 |
|---|---|---|
| committer | Richard Sandiford <rsandifo@gcc.gnu.org> | 2019-11-08 09:06:50 +0000 |
| commit | be5f7eccf766a4c3a59b821f77ce320001df838f (patch) | |
| tree | ee509b6f5eec1502b7562e6c72ec021a316b84fc /gcc | |
| parent | f1e1ed3314b7c6308f64cbbcf6d1916e239c8e35 (diff) | |
| download | gcc-be5f7eccf766a4c3a59b821f77ce320001df838f.zip gcc-be5f7eccf766a4c3a59b821f77ce320001df838f.tar.gz gcc-be5f7eccf766a4c3a59b821f77ce320001df838f.tar.bz2 | |
Handle POLY_INT_CSTs in declare_return_value
SVE allows variable-length vectors to be returned by value,
which tripped the assert in declare_return_variable.
2019-11-08 Richard Sandiford <richard.sandiford@arm.com>
gcc/
* tree-inline.c (declare_return_variable): Check for poly_int_tree_p
instead of INTEGER_CST.
gcc/testsuite/
* gcc.target/aarch64/sve/acle/general/inline_1.c: New test.
From-SVN: r277956
Diffstat (limited to 'gcc')
| -rw-r--r-- | gcc/ChangeLog | 5 | ||||
| -rw-r--r-- | gcc/testsuite/ChangeLog | 4 | ||||
| -rw-r--r-- | gcc/testsuite/gcc.target/aarch64/sve/acle/general/inline_1.c | 6 | ||||
| -rw-r--r-- | gcc/tree-inline.c | 4 |
4 files changed, 17 insertions, 2 deletions
diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 4b1f408..b50456e 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,8 @@ +2019-11-08 Richard Sandiford <richard.sandiford@arm.com> + + * tree-inline.c (declare_return_variable): Check for poly_int_tree_p + instead of INTEGER_CST. + 2019-11-08 Richard Biener <rguenther@suse.de> PR tree-optimization/92324 diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index 358bbf5..9d5702b 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,3 +1,7 @@ +2019-11-08 Richard Sandiford <richard.sandiford@arm.com> + + * gcc.target/aarch64/sve/acle/general/inline_1.c: New test. + 2019-11-08 Richard Biener <rguenther@suse.de> PR tree-optimization/92324 diff --git a/gcc/testsuite/gcc.target/aarch64/sve/acle/general/inline_1.c b/gcc/testsuite/gcc.target/aarch64/sve/acle/general/inline_1.c new file mode 100644 index 0000000..f736ac3 --- /dev/null +++ b/gcc/testsuite/gcc.target/aarch64/sve/acle/general/inline_1.c @@ -0,0 +1,6 @@ +/* { dg-options "-O2" } */ + +#pragma GCC aarch64 "arm_sve.h" + +static inline svint32_t foo () { return svdup_s32 (32); } +svint32_t bar () { return svadd_x (svptrue_b8 (), foo (), 1); } diff --git a/gcc/tree-inline.c b/gcc/tree-inline.c index 2b8b9ee..9d0acd9 100644 --- a/gcc/tree-inline.c +++ b/gcc/tree-inline.c @@ -3654,7 +3654,7 @@ declare_return_variable (copy_body_data *id, tree return_slot, tree modify_dest, /* ??? If we're assigning to a variable sized type, then we must reuse the destination variable, because we've no good way to create variable sized temporaries at this point. */ - else if (TREE_CODE (TYPE_SIZE_UNIT (caller_type)) != INTEGER_CST) + else if (!poly_int_tree_p (TYPE_SIZE_UNIT (caller_type))) use_it = true; /* If the callee cannot possibly modify MODIFY_DEST, then we can @@ -3689,7 +3689,7 @@ declare_return_variable (copy_body_data *id, tree return_slot, tree modify_dest, } } - gcc_assert (TREE_CODE (TYPE_SIZE_UNIT (callee_type)) == INTEGER_CST); + gcc_assert (poly_int_tree_p (TYPE_SIZE_UNIT (callee_type))); var = copy_result_decl_to_var (result, id); DECL_SEEN_IN_BIND_EXPR_P (var) = 1; |