Reviewers: Sven Panne,

Description:
Fix cctests when VFP is disabled.


[email protected]
BUG=
TEST=


Please review this at https://chromiumcodereview.appspot.com/10820027/

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

Affected files:
  M src/arm/builtins-arm.cc


Index: src/arm/builtins-arm.cc
diff --git a/src/arm/builtins-arm.cc b/src/arm/builtins-arm.cc
index 40949ff35401c2898d14ea23ac00b0d37d1b0685..2079e044affb9abf872015ae8cd609bdfdffc6ef 100644
--- a/src/arm/builtins-arm.cc
+++ b/src/arm/builtins-arm.cc
@@ -1283,7 +1283,10 @@ void Builtins::Generate_NotifyOSR(MacroAssembler* masm) {

 void Builtins::Generate_OnStackReplacement(MacroAssembler* masm) {
   CpuFeatures::TryForceFeatureScope scope(VFP3);
-  ASSERT(CPU::SupportsCrankshaft());
+  if (!CPU::SupportsCrankshaft()) {
+    __ Abort("Unreachable code: Cannot optimize without VFP3 support.");
+    return;
+  }

   // Lookup the function in the JavaScript frame and push it as an
   // argument to the on-stack replacement function.


--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev

Reply via email to