diff options
| author | Carl Love <cel@us.ibm.com> | 2018-04-20 15:18:24 +0000 |
|---|---|---|
| committer | Carl Love <carll@gcc.gnu.org> | 2018-04-20 15:18:24 +0000 |
| commit | a3b82e156de228bbd33c87999f1fbdd519140734 (patch) | |
| tree | 358f43fc4b86f5b37603374d65e32bcd5934f1cb | |
| parent | e91eba31fdc49d928090a9b0424247fd6029f044 (diff) | |
| download | gcc-a3b82e156de228bbd33c87999f1fbdd519140734.zip gcc-a3b82e156de228bbd33c87999f1fbdd519140734.tar.gz gcc-a3b82e156de228bbd33c87999f1fbdd519140734.tar.bz2 | |
re PR target/83402 (PPC64 implementation of ./rs6000/emmintrin.h gives out of range for _mm_slli_epi32)
gcc/ChangeLog:
2018-04-20 Carl Love <cel@us.ibm.com>
PR target/83402
* config/rs6000/rs6000-c.c (rs6000_gimple_fold_builtin): Add
size check for arg0.
From-SVN: r259524
| -rw-r--r-- | gcc/ChangeLog | 6 | ||||
| -rw-r--r-- | gcc/config/rs6000/rs6000.c | 17 |
2 files changed, 21 insertions, 2 deletions
diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 7152a19..1702d95 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,9 @@ +2018-04-20 Carl Love <cel@us.ibm.com> + + PR target/83402 + * config/rs6000/rs6000-c.c (rs6000_gimple_fold_builtin): Add + size check for arg0. + 2018-04-20 Nathan Sidwell <nathan@codesourcery.com> Tom de Vries <tom@codesourcery.com> diff --git a/gcc/config/rs6000/rs6000.c b/gcc/config/rs6000/rs6000.c index fd11407..7ba908e 100644 --- a/gcc/config/rs6000/rs6000.c +++ b/gcc/config/rs6000/rs6000.c @@ -16594,10 +16594,23 @@ rs6000_gimple_fold_builtin (gimple_stmt_iterator *gsi) case ALTIVEC_BUILTIN_VSPLTISH: case ALTIVEC_BUILTIN_VSPLTISW: { + int size; + + if (fn_code == ALTIVEC_BUILTIN_VSPLTISB) + size = 8; + else if (fn_code == ALTIVEC_BUILTIN_VSPLTISH) + size = 16; + else + size = 32; + arg0 = gimple_call_arg (stmt, 0); lhs = gimple_call_lhs (stmt); - /* Only fold the vec_splat_*() if arg0 is constant. */ - if (TREE_CODE (arg0) != INTEGER_CST) + + /* Only fold the vec_splat_*() if the lower bits of arg 0 is a + 5-bit signed constant in range -16 to +15. */ + if (TREE_CODE (arg0) != INTEGER_CST + || !IN_RANGE (sext_hwi(TREE_INT_CST_LOW (arg0), size), + -16, 15)) return false; gimple_seq stmts = NULL; location_t loc = gimple_location (stmt); |