[PATCH] D24467: Fix an error after D21678

Akira Hatanaka via cfe-commits cfe-commits at lists.llvm.org
Thu Sep 15 12:51:15 PDT 2016


ahatanak added a comment.

LGTM with a nit in test case.


================
Comment at: llvm/tools/clang/test/CodeGen/vecshift.c:43
@@ +42,3 @@
+  vi8 = 1 << vi8;
+// CHECK: [[t2:%.+]] = load <8 x i32>, <8 x i32>* @vi8, align 32
+// CHECK: shl <8 x i32> <i32 1, i32 1, i32 1, i32 1, i32 1, i32 1, i32 1, i32 1>, [[t2]]
----------------
This test fails on my machine because MaxVectorAlign is 128 (16B) for darwin. Maybe you can remove the alignment checks or add a triple to the RUN line?


https://reviews.llvm.org/D24467





More information about the cfe-commits mailing list