re PR target/78794 (We noticed ~9% regression in 32-bit mode for 462.libquntum on...

	PR target/78794
	* config/i386/i386.c (dimode_scalar_chain::compute_convert_gain):
	Calculate additional gain for andnot for targets without BMI.

testsuite/ChangeLog:

	PR target/78794
	* gcc.target/i386/pr78794.c: New test.

From-SVN: r243615

diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index c946865ee72dca628db690471b348c2103ac70e5..cbf7b2f466d90fe9ec4d27f20ce16d219229550a 100644 (file)
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,9 @@
+2016-12-13  Uros Bizjak  <ubizjak@gmail.com>
+
+       PR target/78794
+       * config/i386/i386.c (dimode_scalar_chain::compute_convert_gain):
+       Calculate additional gain for andnot for targets without BMI.
+
 2016-12-13  Carl Love  <cel@us.ibm.com>
 
        * config/rs6000/rs6000-c.c: Add built-in support for
@@ -21,11 +27,9 @@
 2016-12-13  Martin Liska  <mliska@suse.cz>
 
        PR tree-optimization/78428
-       * expr.c (store_constructor_field): Add new arguments to the
-       function.
-       (store_constructor): Set up bitregion_end and add
-       gcc_unreachable to fields that have either non-constant size
-       or (and) offset.
+       * expr.c (store_constructor_field): Add new arguments to the function.
+       (store_constructor): Set up bitregion_end and add gcc_unreachable to
+       fields that have either non-constant size or (and) offset.
 
 2016-12-13  Marek Polacek  <polacek@redhat.com>
 

diff --git a/gcc/config/i386/i386.c b/gcc/config/i386/i386.c
index 152e6bcafd5d9a8129d7fee002c61096a3ca3d04..8fca3692a311f78446cdcae382a5f02c82ce7052 100644 (file)
--- a/gcc/config/i386/i386.c
+++ b/gcc/config/i386/i386.c
@@ -3419,6 +3419,11 @@ dimode_scalar_chain::compute_convert_gain ()
               || GET_CODE (src) == AND)
        {
          gain += ix86_cost->add;
+         /* Additional gain for andnot for targets without BMI.  */
+         if (GET_CODE (XEXP (src, 0)) == NOT
+             && !TARGET_BMI)
+           gain += 2 * ix86_cost->add;
+
          if (CONST_INT_P (XEXP (src, 0)))
            gain -= vector_const_cost (XEXP (src, 0));
          if (CONST_INT_P (XEXP (src, 1)))
@@ -3431,7 +3436,7 @@ dimode_scalar_chain::compute_convert_gain ()
        {
          /* Assume comparison cost is the same.  */
        }
-      else if (GET_CODE (src) == CONST_INT)
+      else if (CONST_INT_P (src))
        {
          if (REG_P (dst))
            gain += COSTS_N_INSNS (2);

diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog
index 48ad430720bd6c2c68bba67d55f6a7db8c5f77a7..072e617415ac6af328abd1a3acd0786150f4ce20 100644 (file)
--- a/gcc/testsuite/ChangeLog
+++ b/gcc/testsuite/ChangeLog
@@ -1,3 +1,8 @@
+2016-12-13  Uros Bizjak  <ubizjak@gmail.com>
+
+       PR target/78794
+       * gcc.target/i386/pr78794.c: New test.
+
 2016-12-13  Andre Vehreschild  <vehre@gcc.gnu.org>
 
        PR fortran/77785

diff --git a/gcc/testsuite/gcc.target/i386/pr78794.c b/gcc/testsuite/gcc.target/i386/pr78794.c
new file mode 100644 (file)
index 0000000..6c3a3fe
--- /dev/null
+++ b/gcc/testsuite/gcc.target/i386/pr78794.c
@@ -0,0 +1,32 @@
+/* PR target/pr78794 */
+/* { dg-do compile { target { ia32 } } } */
+/* { dg-options "-O2 -march=slm -mno-bmi -mno-stackrealign" } */
+/* { dg-final { scan-assembler "pandn" } } */
+
+typedef unsigned long long ull;
+
+struct S1
+{
+  float x;
+  ull y;
+};
+
+
+struct S2
+{
+  int a1;
+  struct S1 *node;
+  int *a2;
+};
+
+void
+foo(int c1, int c2, int c3, struct S2 *reg)
+{
+  int i;
+  for(i=0; i<reg->a1; i++)
+    if(reg->node[i].y & ((ull) 1 << c1))
+      {
+       if(reg->node[i].y & ((ull) 1 << c2))
+         reg->node[i].y ^= ((ull) 1 << c3);
+      }
+}