[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