Revision: 13542
Author: [email protected]
Date: Tue Jan 29 07:28:05 2013
Log: Fix Mac problem with stubs initialization
[email protected]
Review URL: https://codereview.chromium.org/12096040
http://code.google.com/p/v8/source/detail?r=13542
Modified:
/branches/bleeding_edge/src/arm/lithium-codegen-arm.cc
/branches/bleeding_edge/src/ia32/lithium-codegen-ia32.cc
/branches/bleeding_edge/src/isolate.cc
/branches/bleeding_edge/src/parser.cc
=======================================
--- /branches/bleeding_edge/src/arm/lithium-codegen-arm.cc Mon Jan 28
06:50:47 2013
+++ /branches/bleeding_edge/src/arm/lithium-codegen-arm.cc Tue Jan 29
07:28:05 2013
@@ -66,8 +66,6 @@
ASSERT(is_unused());
status_ = GENERATING;
- CodeStub::GenerateFPStubs();
-
// Open a frame scope to indicate that there is a frame on the stack.
The
// NONE indicates that the scope shouldn't actually generate code to set
up
// the frame (that is done in GeneratePrologue).
=======================================
--- /branches/bleeding_edge/src/ia32/lithium-codegen-ia32.cc Tue Jan 29
02:18:15 2013
+++ /branches/bleeding_edge/src/ia32/lithium-codegen-ia32.cc Tue Jan 29
07:28:05 2013
@@ -72,8 +72,6 @@
ASSERT(is_unused());
status_ = GENERATING;
- CodeStub::GenerateFPStubs();
-
// Open a frame scope to indicate that there is a frame on the stack.
The
// MANUAL indicates that the scope shouldn't actually generate code to
set up
// the frame (that is done in GeneratePrologue).
=======================================
--- /branches/bleeding_edge/src/isolate.cc Tue Jan 29 04:50:42 2013
+++ /branches/bleeding_edge/src/isolate.cc Tue Jan 29 07:28:05 2013
@@ -2098,6 +2098,7 @@
if (!Serializer::enabled()) {
// Ensure that the stub failure trampoline has been generated.
HandleScope scope(this);
+ CodeStub::GenerateFPStubs();
StubFailureTrampolineStub().GetCode();
}
=======================================
--- /branches/bleeding_edge/src/parser.cc Wed Jan 9 02:30:54 2013
+++ /branches/bleeding_edge/src/parser.cc Tue Jan 29 07:28:05 2013
@@ -52,7 +52,10 @@
class PositionStack {
public:
explicit PositionStack(bool* ok) : top_(NULL), ok_(ok) {}
- ~PositionStack() { ASSERT(!*ok_ || is_empty()); }
+ ~PositionStack() {
+ ASSERT(!*ok_ || is_empty());
+ USE(ok_);
+ }
class Element {
public:
--
--
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/groups/opt_out.
