[PATCH] [X86] replace vextractf128 intrinsics with generic shuffles

Craig Topper craig.topper at gmail.com
Thu Mar 12 09:54:58 PDT 2015


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/






More information about the cfe-commits mailing list