[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