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

[Richard Smith via cfe-commits]

rsmith added inline comments.

================
Comment at: lib/Basic/Targets.cpp:2548
@@ +2547,3 @@
+  // All X86 processors but i386 have X87.
+  if (Kind != CK_i386)
+    setFeatureEnabledImpl(Features, "x87", true);
----------------
What about `CK_Generic`? Also, if `CK_i486` can be used for the 486SX, we need to exclude it here too. It looks (from wikipedia) like all the "WinChip" flavours of 486 have an x87 unit, but it'd be nice if someone could confirm that. Maybe we should have separate `CPUKind`s for 486 SX versus 486 DX.


http://reviews.llvm.org/D13980