diff options
| author | Andreas Krebbel <krebbel@linux.ibm.com> | 2018-10-15 08:07:13 +0000 |
|---|---|---|
| committer | Andreas Krebbel <krebbel@gcc.gnu.org> | 2018-10-15 08:07:13 +0000 |
| commit | cab5fe166dd13e1668801f00189459d3139a9342 (patch) | |
| tree | 1d4e7ac2f5c4d261248def483663d763211cb758 | |
| parent | 4b186707ee29b3189728731adf0ebdd2e620a26d (diff) | |
| download | gcc-cab5fe166dd13e1668801f00189459d3139a9342.zip gcc-cab5fe166dd13e1668801f00189459d3139a9342.tar.gz gcc-cab5fe166dd13e1668801f00189459d3139a9342.tar.bz2 | |
S/390: Fix problem with vec_init expander
gcc/ChangeLog:
2018-10-15 Andreas Krebbel <krebbel@linux.ibm.com>
* config/s390/s390.c (s390_expand_vec_init): Force vector element
into reg if it isn't a general operand.
gcc/testsuite/ChangeLog:
2018-10-15 Andreas Krebbel <krebbel@linux.ibm.com>
* g++.dg/vec-init-1.C: New test.
From-SVN: r265158
| -rw-r--r-- | gcc/ChangeLog | 5 | ||||
| -rw-r--r-- | gcc/config/s390/s390.c | 11 | ||||
| -rw-r--r-- | gcc/testsuite/ChangeLog | 4 | ||||
| -rw-r--r-- | gcc/testsuite/g++.dg/vec-init-1.C | 26 |
4 files changed, 43 insertions, 3 deletions
diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 8144181..4f33576 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,8 @@ +2018-10-15 Andreas Krebbel <krebbel@linux.ibm.com> + + * config/s390/s390.c (s390_expand_vec_init): Force vector element + into reg if it isn't a general operand. + 2018-10-14 H.J. Lu <hongjiu.lu@intel.com> PR target/87599 diff --git a/gcc/config/s390/s390.c b/gcc/config/s390/s390.c index 71039fe..ab22c2c 100644 --- a/gcc/config/s390/s390.c +++ b/gcc/config/s390/s390.c @@ -6627,11 +6627,16 @@ s390_expand_vec_init (rtx target, rtx vals) return; } + /* Use vector replicate instructions. vlrep/vrepi/vrep */ if (all_same) { - emit_insn (gen_rtx_SET (target, - gen_rtx_VEC_DUPLICATE (mode, - XVECEXP (vals, 0, 0)))); + rtx elem = XVECEXP (vals, 0, 0); + + /* vec_splats accepts general_operand as source. */ + if (!general_operand (elem, GET_MODE (elem))) + elem = force_reg (inner_mode, elem); + + emit_insn (gen_rtx_SET (target, gen_rtx_VEC_DUPLICATE (mode, elem))); return; } diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index 8824474..f12a743 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,3 +1,7 @@ +2018-10-15 Andreas Krebbel <krebbel@linux.ibm.com> + + * g++.dg/vec-init-1.C: New test. + 2018-10-14 H.J. Lu <hongjiu.lu@intel.com> PR target/87599 diff --git a/gcc/testsuite/g++.dg/vec-init-1.C b/gcc/testsuite/g++.dg/vec-init-1.C new file mode 100644 index 0000000..f35d39c --- /dev/null +++ b/gcc/testsuite/g++.dg/vec-init-1.C @@ -0,0 +1,26 @@ +/* On S/390 this ends up calling the vec_init RTL expander with a + parallel of two symbol_refs. */ + +/* { dg-do compile } */ +/* { dg-options "-O3 -fPIC" } */ + + +struct test +{ + struct base + { + int key; + }; + struct derived : public base + { + int key; + }; + + derived core; + derived &dRef; + base &bRef; + + test() : dRef (core), bRef (core) {} +}; + +test test; |