Change permute index type to unsigned short

This patch changes the element type of (auto_)vec_perm_indices from
unsigned char to unsigned short.  This is needed for fixed-length
2048-bit SVE.  (SVE is variable-length by default, but it's possible
to ask for specific vector lengths if you want to.)

2017-09-25  Richard Sandiford  <richard.sandiford@linaro.org>

gcc/
	* target.h (vec_perm_indices): Use unsigned short rather than
	unsigned char.
	(auto_vec_perm_indices): Likewise.
	* config/aarch64/aarch64.c (aarch64_vectorize_vec_perm_const_ok):
	Use unsigned int rather than unsigned char.
	* config/arm/arm.c (arm_vectorize_vec_perm_const_ok): Likewise.

From-SVN: r253150

diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index f4060b103bf3353a082e17a16496f2255477dfc7..b53da5390d564b052b6d6da8f62c89c0f40ea378 100644 (file)
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,12 @@
+2017-09-25  Richard Sandiford  <richard.sandiford@linaro.org>
+
+       * target.h (vec_perm_indices): Use unsigned short rather than
+       unsigned char.
+       (auto_vec_perm_indices): Likewise.
+       * config/aarch64/aarch64.c (aarch64_vectorize_vec_perm_const_ok):
+       Use unsigned int rather than unsigned char.
+       * config/arm/arm.c (arm_vectorize_vec_perm_const_ok): Likewise.
+
 2017-09-25  Richard Biener  <rguenther@suse.de>
 
        * cfgloop.h (sort_sibling_loops): Declare.

diff --git a/gcc/config/aarch64/aarch64.c b/gcc/config/aarch64/aarch64.c
index f38540b396aaf80d5c022a977977614673ec3725..9b02c6e85e4a10932c2fb49a6637e29ed5fb0115 100644 (file)
--- a/gcc/config/aarch64/aarch64.c
+++ b/gcc/config/aarch64/aarch64.c
@@ -13820,7 +13820,7 @@ aarch64_vectorize_vec_perm_const_ok (machine_mode vmode, vec_perm_indices sel)
   nelt = sel.length ();
   for (i = which = 0; i < nelt; ++i)
     {
-      unsigned char e = d.perm[i];
+      unsigned int e = d.perm[i];
       gcc_assert (e < 2 * nelt);
       which |= (e < nelt ? 1 : 2);
     }

diff --git a/gcc/config/arm/arm.c b/gcc/config/arm/arm.c
index 3671616cfa7adbaf8d7aa1e0ee582a881bf3c0ea..f9c1ce35f37dad962ab8b6cdfb407bb836fce599 100644 (file)
--- a/gcc/config/arm/arm.c
+++ b/gcc/config/arm/arm.c
@@ -29261,7 +29261,7 @@ arm_vectorize_vec_perm_const_ok (machine_mode vmode, vec_perm_indices sel)
   nelt = GET_MODE_NUNITS (d.vmode);
   for (i = which = 0; i < nelt; ++i)
     {
-      unsigned char e = d.perm[i];
+      unsigned int e = d.perm[i];
       gcc_assert (e < 2 * nelt);
       which |= (e < nelt ? 1 : 2);
     }

diff --git a/gcc/target.h b/gcc/target.h
index 64e1d68f0e01a7620df16b427957d3e9ab650915..ac43b16a549cf6efba7a9372bfb2d62355254c81 100644 (file)
--- a/gcc/target.h
+++ b/gcc/target.h
@@ -193,11 +193,11 @@ enum vect_cost_model_location {
 
 /* The type to use for vector permutes with a constant permute vector.
    Each entry is an index into the concatenated input vectors.  */
-typedef vec<unsigned char> vec_perm_indices;
+typedef vec<unsigned short> vec_perm_indices;
 
 /* Same, but can be used to construct local permute vectors that are
    automatically freed.  */
-typedef auto_vec<unsigned char, 32> auto_vec_perm_indices;
+typedef auto_vec<unsigned short, 32> auto_vec_perm_indices;
 
 /* The target structure.  This holds all the backend hooks.  */
 #define DEFHOOKPOD(NAME, DOC, TYPE, INIT) TYPE NAME;