REPOSITORY
  rL LLVM

================
Comment at: lib/Headers/avxintrin.h:1184
@@ +1183,3 @@
+    (__v8sf)(V), \
+    (__v8sf)(V), \
+    (((M) & 1) ? 4 : 0), \
----------------
Not sure if its safe to access V twice. It could be a function call that we 
maybe shouldn't invoke twice. May need a temporary here even though it might 
fail Wshadow

Or maybe just feed an appropriate zero vector to the second shuffle argument?

http://reviews.llvm.org/D8275

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/



_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to