[llvm] r271482 - [X86] No need to use 256-bit VMOVNTPS for integer types when only AVX1 is supported. VMOVNTDQ is available with AVX1.

[Craig Topper via llvm-commits]

Author: ctopper
Date: Wed Jun  1 23:19:48 2016
New Revision: 271482

URL: http://llvm.org/viewvc/llvm-project?rev=271482&view=rev
Log:
[X86] No need to use 256-bit VMOVNTPS for integer types when only AVX1 is supported. VMOVNTDQ is available with AVX1.

We were getting this right for v4i64 but not the other integer types.

Modified:
    llvm/trunk/lib/Target/X86/X86InstrSSE.td

Modified: llvm/trunk/lib/Target/X86/X86InstrSSE.td
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/X86/X86InstrSSE.td?rev=271482&r1=271481&r2=271482&view=diff
==============================================================================
--- llvm/trunk/lib/Target/X86/X86InstrSSE.td (original)
+++ llvm/trunk/lib/Target/X86/X86InstrSSE.td Wed Jun  1 23:19:48 2016
@@ -3631,28 +3631,14 @@ def MOVNTI_64mr : RI<0xC3, MRMDestMem, (
                   PS, Requires<[HasSSE2]>;
 } // SchedRW = [WriteStore]
 
-let Predicates = [HasAVX2, NoVLX] in {
+let Predicates = [HasAVX, NoVLX] in {
   def : Pat<(alignednontemporalstore (v8i32 VR256:$src), addr:$dst),
             (VMOVNTDQYmr addr:$dst, VR256:$src)>;
   def : Pat<(alignednontemporalstore (v16i16 VR256:$src), addr:$dst),
             (VMOVNTDQYmr addr:$dst, VR256:$src)>;
   def : Pat<(alignednontemporalstore (v32i8 VR256:$src), addr:$dst),
             (VMOVNTDQYmr addr:$dst, VR256:$src)>;
-}
-
-// If integer type isn't available, use the floating point type.
-let Predicates = [HasAVX1Only] in {
-  def : Pat<(alignednontemporalstore (v4i64 VR256:$src), addr:$dst),
-            (VMOVNTPSYmr addr:$dst, VR256:$src)>;
-  def : Pat<(alignednontemporalstore (v8i32 VR256:$src), addr:$dst),
-            (VMOVNTPSYmr addr:$dst, VR256:$src)>;
-  def : Pat<(alignednontemporalstore (v16i16 VR256:$src), addr:$dst),
-            (VMOVNTPSYmr addr:$dst, VR256:$src)>;
-  def : Pat<(alignednontemporalstore (v32i8 VR256:$src), addr:$dst),
-            (VMOVNTPSYmr addr:$dst, VR256:$src)>;
-}
 
-let Predicates = [HasAVX, NoVLX] in {
   def : Pat<(alignednontemporalstore (v4i32 VR128:$src), addr:$dst),
             (VMOVNTDQmr addr:$dst, VR128:$src)>;
   def : Pat<(alignednontemporalstore (v8i16 VR128:$src), addr:$dst),