[PATCH] D20782: [AVX512] Emit generic masked store intrinsics directly from clang instead of using x86 specific intrinsics.
Elena Demikhovsky via cfe-commits
cfe-commits at lists.llvm.org
Sun May 29 22:39:55 PDT 2016
delena added inline comments.
================
Comment at: lib/CodeGen/CGBuiltin.cpp:6304
@@ +6303,3 @@
+ Indices[i] = i;
+ Ops[2] = CGF.Builder.CreateShuffleVector(Ops[2], Ops[2],
+ makeArrayRef(Indices, NumElts),
----------------
What code do you receive at the end? There is no shuffle instruction in the architecture for mask vector.
================
Comment at: test/CodeGen/avx512f-builtins.c:123
@@ -122,2 +122,3 @@
// CHECK-LABEL: @test_mm512_storeu_si512
- // CHECK: @llvm.x86.avx512.mask.storeu.d.512
+ // CHECK: store <16 x i32> %5, <16 x i32>* %6, align 1
+ // CHECK-NEXT: ret void
----------------
I suggest to remove %5, %6 from the test, you can put something like this:
CHECK: store <16 x i32> {{.*}}, align 1
http://reviews.llvm.org/D20782
More information about the cfe-commits
mailing list