RKSimon added a comment.

Definitely like this idea - hopefully it'll make it safer to remove unnecessary 
builtins.


================
Comment at: avx2-builtins-codegen.c:182
@@ -151,1 +181,3 @@
+  // FIXME-CHECK-ASM: vpxor %ymm{{.*}}
+  // FIXME-CHECK-ASM: vandps {{.*}}, %ymm{{.*}}
   return _mm256_andnot_si256(a, b);
----------------
Please can you add a comment explaining the FIXME?


http://reviews.llvm.org/D12212



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to