[PATCH] D13980: Add "x87" in x86 target feature map

Eric Christopher via cfe-commits cfe-commits at lists.llvm.org
Thu Nov 19 13:26:46 PST 2015


echristo added a subscriber: echristo.
echristo added a comment.

Are there any of the intrinsics in the headers that also depend on x87?

One inline comment.

-eric


================
Comment at: lib/Basic/Targets.cpp:2538-2539
@@ -2537,1 +2537,4 @@
 
+  // All X86 processors but i386 have X87.
+  if (getCPUKind(CPU) != CK_i386)
+    setFeatureEnabledImpl(Features, "x87", true);
----------------
aturetsk wrote:
> You are right. Fixed.
Mind only doing getCPUKind once here? :)


http://reviews.llvm.org/D13980





More information about the cfe-commits mailing list