Reviewers: Sven Panne,

Description:
Remove check for SAHF support in ia32.

On ia32, SAHF is guaranteed. It's only necessary to probe for it on x64.

[email protected]
BUG=376087

Please review this at https://codereview.chromium.org/296133006/

SVN Base: https://v8.googlecode.com/svn/branches/bleeding_edge

Affected files (+0, -2 lines):
  M src/ia32/assembler-ia32.cc


Index: src/ia32/assembler-ia32.cc
diff --git a/src/ia32/assembler-ia32.cc b/src/ia32/assembler-ia32.cc
index fea67e3c7b24280ede21a9df3a7d6aec72e861cc..cee926ffb04e12ab2fc9bac1e9cbf879ca0a0f1c 100644
--- a/src/ia32/assembler-ia32.cc
+++ b/src/ia32/assembler-ia32.cc
@@ -52,8 +52,6 @@ void CpuFeatures::ProbeImpl(bool cross_compile) {
   CPU cpu;
   CHECK(cpu.has_sse2());  // SSE2 support is mandatory.
   CHECK(cpu.has_cmov());  // CMOV support is mandatory.
-  CHECK(cpu.has_sahf());  // SAHF must be available in compat/legacy mode.
-  supported_ |= 1u << SAHF;
   supported_ |= OS::CpuFeaturesImpliedByPlatform();

   // Only use statically determined features for cross compile (snapshot).


--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
--- You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
For more options, visit https://groups.google.com/d/optout.

Reply via email to