Reviewers: Weiliang,

Message:
PTAL

Description:
X87: initialize the FPU state for X87 in prologue.

   This CL is a fix for c0c3d866fb824290c24d4c2a14414019d844515f (r30606).
   In r30606, initialization of FPU implementation is not moved to prologue
   generation correctly.

BUG=

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

Base URL: https://chromium.googlesource.com/v8/v8.git@master

Affected files (+4, -3 lines):
  M src/x87/lithium-codegen-x87.cc


Index: src/x87/lithium-codegen-x87.cc
diff --git a/src/x87/lithium-codegen-x87.cc b/src/x87/lithium-codegen-x87.cc
index d9e5169fdec0dc6f60efaa20d55c1045e707daf8..1b5b176be134ad9034a0d0a9032ed05bc7e2c6a3 100644
--- a/src/x87/lithium-codegen-x87.cc
+++ b/src/x87/lithium-codegen-x87.cc
@@ -215,6 +215,10 @@ bool LCodeGen::GeneratePrologue() {
       }
     }
   }
+
+  // Initailize FPU state.
+  __ fninit();
+
   return !is_aborted();
 }

@@ -278,9 +282,6 @@ void LCodeGen::DoPrologue(LPrologue* instr) {
     Comment(";;; End allocate local context");
   }

-  // Initailize FPU state.
-  __ fninit();
-
   Comment(";;; Prologue end");
 }



--
--
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