This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] Fix fold-const BIT_FIELD_REF folding with 1 elt vectors (PR tree-optimization/85467)
- From: Richard Biener <rguenther at suse dot de>
- To: Jakub Jelinek <jakub at redhat dot com>
- Cc: gcc-patches at gcc dot gnu dot org
- Date: Thu, 19 Apr 2018 20:45:31 +0200
- Subject: Re: [PATCH] Fix fold-const BIT_FIELD_REF folding with 1 elt vectors (PR tree-optimization/85467)
- References: <20180419183313.GB8577@tucnak>
On April 19, 2018 8:33:13 PM GMT+02:00, Jakub Jelinek <jakub@redhat.com> wrote:
>Hi!
>
>This PR is the fold-const.c counterpart of the match.pd PR85195,
>we first check if the result type is either the element type or vector
>type
>with the same element type, and then for extraction of a single element
>simply assume that the result must be the element type; it could be
>single
>element vector with that element type too though.
>
>Fixed thusly, bootstrapped/regtested on x86_64-linux and i686-linux, ok
>for
>trunk?
OK.
Richard.
>2018-04-19 Jakub Jelinek <jakub@redhat.com>
>
> PR tree-optimization/85467
> * fold-const.c (fold_ternary_loc) <case BIT_FIELD_REF>: Use
> VECTOR_TYPE_P macro. If type is vector type, VIEW_CONVERT_EXPR the
> VECTOR_CST element to type.
>
> * gcc.dg/pr85467.c: New test.
>
>--- gcc/fold-const.c.jj 2018-04-11 12:20:53.000000000 +0200
>+++ gcc/fold-const.c 2018-04-19 14:34:30.410710684 +0200
>@@ -11631,7 +11631,7 @@ fold_ternary_loc (location_t loc, enum t
> case BIT_FIELD_REF:
> if (TREE_CODE (arg0) == VECTOR_CST
> && (type == TREE_TYPE (TREE_TYPE (arg0))
>- || (TREE_CODE (type) == VECTOR_TYPE
>+ || (VECTOR_TYPE_P (type)
> && TREE_TYPE (type) == TREE_TYPE (TREE_TYPE (arg0))))
> && tree_fits_uhwi_p (op1)
> && tree_fits_uhwi_p (op2))
>@@ -11653,7 +11653,12 @@ fold_ternary_loc (location_t loc, enum t
> if (TREE_CODE (arg0) == VECTOR_CST)
> {
> if (n == 1)
>- return VECTOR_CST_ELT (arg0, idx);
>+ {
>+ tem = VECTOR_CST_ELT (arg0, idx);
>+ if (VECTOR_TYPE_P (type))
>+ tem = fold_build1 (VIEW_CONVERT_EXPR, type, tem);
>+ return tem;
>+ }
>
> tree_vector_builder vals (type, n, 1);
> for (unsigned i = 0; i < n; ++i)
>--- gcc/testsuite/gcc.dg/pr85467.c.jj 2018-04-19 14:52:53.629273520
>+0200
>+++ gcc/testsuite/gcc.dg/pr85467.c 2018-04-19 14:51:57.348243306 +0200
>@@ -0,0 +1,30 @@
>+/* PR tree-optimization/85467 */
>+/* { dg-do compile } */
>+/* { dg-options "-O2 -fno-tree-ccp --param=sccvn-max-scc-size=10" } */
>+
>+#define TEST(N, T) \
>+typedef T V##N __attribute__ ((__vector_size__ (sizeof (T)))); \
>+ \
>+V##N \
>+bar##N (V##N u, V##N v) \
>+{ \
>+ do \
>+ v *= (T)((V##N){}[0] ? u[v[0]] : 0); \
>+ while ((V##N){}[0]); \
>+ return v; \
>+} \
>+ \
>+void \
>+foo##N (void) \
>+{ \
>+ bar##N ((V##N){}, (V##N){}); \
>+}
>+
>+TEST (1, char)
>+TEST (2, short)
>+TEST (3, int)
>+TEST (4, long)
>+TEST (5, long long)
>+#ifdef __SIZEOF_INT128__
>+TEST (6, __int128)
>+#endif
>
> Jakub