Updates refactoring of CPU feature detection

runtime/vm/cpu_ia32.cc

Index: runtime/vm/cpu_ia32.cc
===================================================================
--- runtime/vm/cpu_ia32.cc	(revision 32743)
+++ runtime/vm/cpu_ia32.cc	(working copy)
@@ -3,18 +3,21 @@
 // BSD-style license that can be found in the LICENSE file.
 
 #include "vm/globals.h"
-
 #if defined(TARGET_ARCH_IA32)
 
 #include "vm/cpu.h"
 
+#include "vm/assembler.h"
 #include "vm/constants_ia32.h"
+#include "vm/cpuinfo.h"
 #include "vm/heap.h"
 #include "vm/isolate.h"
 #include "vm/object.h"
 
 namespace dart {
 
+DEFINE_FLAG(bool, use_sse41, true, "Use SSE 4.1 if available");
+
 void CPU::FlushICache(uword start, uword size) {
   // Nothing to be done here.
 }
@@ -24,6 +27,33 @@
   return "ia32";
 }
 
+
+bool HostCPUFeatures::sse2_supported_ = false;
+bool HostCPUFeatures::sse4_1_supported_ = false;
+const char* HostCPUFeatures::hardware_ = NULL;
+#ifdef DEBUG
+bool HostCPUFeatures::initialized_ = false;
+#endif
+
+
+#define __ assembler.

[siva, 2014/02/21 19:28:55]

I don't see any use of __ why is it defined here and undefed immediately
afterwards.

[zra, 2014/02/21 21:51:55]

Removed.

+
+void HostCPUFeatures::InitOnce() {
+  CpuInfo::InitOnce(kCpuInfoDefault);
+
+  hardware_ = CpuInfo::GetCpuModel();
+  sse2_supported_ = CpuInfo::FieldContains(kCpuInfoFeatures, "sse2");
+  sse4_1_supported_ =
+      CpuInfo::FieldContains(kCpuInfoFeatures, "sse4_1") ||
+      CpuInfo::FieldContains(kCpuInfoFeatures, "sse4.1");
+
+#ifdef DEBUG
+  initialized_ = true;
+#endif
+}
+
+#undef __
+
 }  // namespace dart
 
 #endif  // defined TARGET_ARCH_IA32