[PATCH] Loop Vectorizer doesn't use %zmm registers on targets supporting AVX512.

[Arnold Schwaighofer]

  Curious, did you test whether the cost model works reasonably well across the test-suite with this turned on? Are there any/many major regressions with this patch applied vs without on an avx512 architecture?


  Thanks,
  Arnold


================
Comment at: test/CodeGen/X86/avx512-vectorizer.ll:8
@@ +7,3 @@
+
+define void @foo(float* noalias nocapture readonly %a, float* noalias nocapture readonly %b, float* noalias nocapture %c, i32 %n) #0 {
+entry:
----------------
You could remove the no capture and readonly attributes. They are not needed for this test case.

I am also surprised that llvm accepts dangling function attributes ("#0”) but it does :).


http://llvm-reviews.chandlerc.com/D3078