From: Eric Christopher Date: Fri, 19 Jun 2015 18:09:33 +0000 (+0000) Subject: Add a comment and FIXME based on the commit message that made the X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=b33e4990d019ce591152189d9f39eea34e0059fb;p=clang Add a comment and FIXME based on the commit message that made the intrinsic _mm_prefetch into a builtin rather than by textual inclusion via the intrinsic headers. git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@240152 91177308-0d34-0410-b5e6-96231b3b80d8 --- diff --git a/include/clang/Basic/BuiltinsX86.def b/include/clang/Basic/BuiltinsX86.def index 1a597b5c27..00c1340ac3 100644 --- a/include/clang/Basic/BuiltinsX86.def +++ b/include/clang/Basic/BuiltinsX86.def @@ -59,6 +59,9 @@ BUILTIN(__builtin_ia32_pswapdsi, "V2iV2i", "nc") // All MMX instructions will be generated via builtins. Any MMX vector // types (<1 x i64>, <2 x i32>, etc.) that aren't used by these builtins will be // expanded by the back-end. +// FIXME: _mm_prefetch must be a built-in because it takes a compile-time constant +// argument and our prior approach of using a #define to the current built-in +// doesn't work in the presence of re-declaration of _mm_prefetch for windows. BUILTIN(_mm_prefetch, "vcC*i", "nc") BUILTIN(__builtin_ia32_emms, "v", "") BUILTIN(__builtin_ia32_paddb, "V8cV8cV8c", "")