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.