aboutsummaryrefslogtreecommitdiff
path: root/gcc
diff options
context:
space:
mode:
authorH.J. Lu <hongjiu.lu@intel.com>2016-08-11 17:36:52 +0000
committerH.J. Lu <hjl@gcc.gnu.org>2016-08-11 10:36:52 -0700
commitc73dd9864bb716800209f7fe5e86d95c4a3e9716 (patch)
tree7d56c0e50e0dd66c8dfb2d8b952c2b03f83dc0fe /gcc
parent385ed708b272bb4b53aeda3081960f63ac56d7b0 (diff)
downloadgcc-c73dd9864bb716800209f7fe5e86d95c4a3e9716.zip
gcc-c73dd9864bb716800209f7fe5e86d95c4a3e9716.tar.gz
gcc-c73dd9864bb716800209f7fe5e86d95c4a3e9716.tar.bz2
Support TImode CONST_WIDE_INT store in 64-bit STV
Support TImode CONST_WIDE_INT store generated from piecewise store. Need to verify performance impact before enabling TImode CONST_INT store for __int128. gcc/ * config/i386/i386.c (timode_scalar_to_vector_candidate_p): Allow TImode CONST_WIDE_INT store. (timode_scalar_chain::convert_insn): Handle CONST_WIDE_INT store. gcc/testsuite/ * gcc.target/i386/pieces-strcpy-1.c: New test. * gcc.target/i386/pieces-strcpy-2.c: Likewise. From-SVN: r239383
Diffstat (limited to 'gcc')
-rw-r--r--gcc/ChangeLog6
-rw-r--r--gcc/config/i386/i386.c23
-rw-r--r--gcc/testsuite/ChangeLog5
-rw-r--r--gcc/testsuite/gcc.target/i386/pieces-strcpy-1.c15
-rw-r--r--gcc/testsuite/gcc.target/i386/pieces-strcpy-2.c15
5 files changed, 61 insertions, 3 deletions
diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index 9ed3a74..97c99c7 100644
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,5 +1,11 @@
2015-08-11 H.J. Lu <hongjiu.lu@intel.com>
+ * config/i386/i386.c (timode_scalar_to_vector_candidate_p): Allow
+ TImode CONST_WIDE_INT store.
+ (timode_scalar_chain::convert_insn): Handle CONST_WIDE_INT store.
+
+2015-08-11 H.J. Lu <hongjiu.lu@intel.com>
+
* config/i386/i386.h (MOVE_MAX_PIECES): Use TImode in 64-bit
mode if unaligned SSE load and store are optimal.
diff --git a/gcc/config/i386/i386.c b/gcc/config/i386/i386.c
index 93eaab1..d086ede 100644
--- a/gcc/config/i386/i386.c
+++ b/gcc/config/i386/i386.c
@@ -2862,9 +2862,12 @@ timode_scalar_to_vector_candidate_p (rtx_insn *insn)
if (MEM_P (dst))
{
- /* Check for store. Only support store from register or standard
- SSE constants. Memory must be aligned or unaligned store is
- optimal. */
+ /* Check for store. Memory must be aligned or unaligned store
+ is optimal. Only support store from register, standard SSE
+ constant or CONST_WIDE_INT generated from piecewise store.
+
+ ??? Verify performance impact before enabling CONST_INT for
+ __int128 store. */
if (misaligned_operand (dst, TImode)
&& !TARGET_SSE_UNALIGNED_STORE_OPTIMAL)
return false;
@@ -2875,6 +2878,7 @@ timode_scalar_to_vector_candidate_p (rtx_insn *insn)
return false;
case REG:
+ case CONST_WIDE_INT:
return true;
case CONST_INT:
@@ -3868,6 +3872,19 @@ timode_scalar_chain::convert_insn (rtx_insn *insn)
PUT_MODE (src, V1TImode);
break;
+ case CONST_WIDE_INT:
+ if (NONDEBUG_INSN_P (insn))
+ {
+ /* Since there are no instructions to store 128-bit constant,
+ temporary register usage is required. */
+ rtx tmp = gen_reg_rtx (V1TImode);
+ src = gen_rtx_CONST_VECTOR (V1TImode, gen_rtvec (1, src));
+ src = validize_mem (force_const_mem (V1TImode, src));
+ emit_conversion_insns (gen_rtx_SET (dst, tmp), insn);
+ dst = tmp;
+ }
+ break;
+
case CONST_INT:
switch (standard_sse_constant_p (src, TImode))
{
diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog
index bb47f76..f33636a 100644
--- a/gcc/testsuite/ChangeLog
+++ b/gcc/testsuite/ChangeLog
@@ -1,3 +1,8 @@
+2015-08-11 H.J. Lu <hongjiu.lu@intel.com>
+
+ * gcc.target/i386/pieces-strcpy-1.c: New test.
+ * gcc.target/i386/pieces-strcpy-2.c: Likewise.
+
2016-08-11 Jakub Jelinek <jakub@redhat.com>
PR c++/72868
diff --git a/gcc/testsuite/gcc.target/i386/pieces-strcpy-1.c b/gcc/testsuite/gcc.target/i386/pieces-strcpy-1.c
new file mode 100644
index 0000000..64b7329
--- /dev/null
+++ b/gcc/testsuite/gcc.target/i386/pieces-strcpy-1.c
@@ -0,0 +1,15 @@
+/* { dg-do compile { target { ! ia32 } } } */
+/* { dg-options "-O2 -mno-avx -msse2 -mtune=generic" } */
+
+extern char *strcpy (char *, const char *);
+
+void
+foo (char *s)
+{
+ strcpy (s,
+ "1234567890abcdef123456abcdef5678123456abcdef567abcdef678"
+ "1234567");
+}
+
+/* { dg-final { scan-assembler-times "movdqa\[ \\t\]+\[^\n\]*%xmm" 4 } } */
+/* { dg-final { scan-assembler-times "movups\[ \\t\]+\[^\n\]*%xmm" 4 } } */
diff --git a/gcc/testsuite/gcc.target/i386/pieces-strcpy-2.c b/gcc/testsuite/gcc.target/i386/pieces-strcpy-2.c
new file mode 100644
index 0000000..7421255
--- /dev/null
+++ b/gcc/testsuite/gcc.target/i386/pieces-strcpy-2.c
@@ -0,0 +1,15 @@
+/* { dg-do compile { target { ! ia32 } } } */
+/* { dg-options "-O2 -mno-avx2 -mavx -mtune=sandybridge" } */
+
+extern char *strcpy (char *, const char *);
+
+void
+foo (char *s)
+{
+ strcpy (s,
+ "1234567890abcdef123456abcdef5678123456abcdef567abcdef678"
+ "1234567");
+}
+
+/* { dg-final { scan-assembler-times "vmovdqa\[ \\t\]+\[^\n\]*%xmm" 4 } } */
+/* { dg-final { scan-assembler-times "vmovups\[ \\t\]+\[^\n\]*%xmm" 4 } } */