[PATCH] D15572: [X86] Add missing m64/int64 conversions

Michael Kuperstein via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 17 05:10:57 PST 2015


mkuper added a comment.

Thanks, Elena.


================
Comment at: lib/Headers/mmintrin.h:446
@@ -445,2 +445,3 @@
 #define _m_from_int _mm_cvtsi32_si64
+#define _m_from_int64 _mm_cvtsi64_m64
 #define _m_to_int _mm_cvtsi64_si32
----------------
delena wrote:
> Under X86_64  or always?
Good question. I mean, _m_to_int64 and _mm_cvtm64_si64 are always equivalent, but perhaps both ought to be available only on x86_64.
In any case, right now _mm_cvtm64_si64 is always available, so I'd rather leave it as is. Possibly both should be desiabled for i686 in a separate commit.


http://reviews.llvm.org/D15572





More information about the llvm-commits mailing list