[PATCH] D37294: [X86][Skylake] Adding the scheduling information for the SkylakeClient target

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 4 10:08:40 PDT 2017


RKSimon added inline comments.


================
Comment at: test/CodeGen/X86/avx2-intrinsics-x86.ll:1625
+; AVX512VL-NEXT:    vmovaps %ymm2, %ymm3 ## EVEX TO VEX Compression encoding: [0xc5,0xfc,0x28,0xda]
+; AVX512VL-NEXT:    vgatherdps %ymm3, (%ecx,%ymm1,4), %ymm0 ## encoding: [0xc4,0xe2,0x65,0x92,0x04,0x89]
 ; AVX512VL-NEXT:    vmovups %ymm2, (%eax) ## EVEX TO VEX Compression encoding: [0xc5,0xfc,0x11,0x10]
----------------
I feels tests like this shouldn't change with schedule model tweaks - it's supposed to be a codegen test not a target specific test.

Would it be possible to reduce the use of -mcpu from tests that aren't actually testing for scheduling? 


Repository:
  rL LLVM

https://reviews.llvm.org/D37294





More information about the llvm-commits mailing list