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

Arnold Schwaighofer aschwaighofer at apple.com
Fri Mar 14 09:08:17 PDT 2014

  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?


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 {
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 :).


More information about the llvm-commits mailing list