Revision: 22281
Author: [email protected]
Date: Tue Jul 8 14:13:50 2014 UTC
Log: Fix issues with code serializer.
- code pre-aging does not work with serializing.
- compilation info needs to remember that we compile for serializing.
- test case leaks memory.
[email protected]
Review URL: https://codereview.chromium.org/379563002
http://code.google.com/p/v8/source/detail?r=22281
Modified:
/branches/bleeding_edge/src/compiler.cc
/branches/bleeding_edge/src/compiler.h
/branches/bleeding_edge/src/full-codegen.cc
/branches/bleeding_edge/test/cctest/cctest.status
/branches/bleeding_edge/test/cctest/test-compiler.cc
=======================================
--- /branches/bleeding_edge/src/compiler.cc Tue Jul 8 09:04:08 2014 UTC
+++ /branches/bleeding_edge/src/compiler.cc Tue Jul 8 14:13:50 2014 UTC
@@ -981,6 +981,9 @@
info.SetCachedData(cached_data, cached_data_mode);
info.SetExtension(extension);
info.SetContext(context);
+ if (FLAG_serialize_toplevel && cached_data_mode ==
PRODUCE_CACHED_DATA) {
+ info.PrepareForSerializing();
+ }
if (FLAG_use_strict) info.SetStrictMode(STRICT);
result = CompileToplevel(&info);
=======================================
--- /branches/bleeding_edge/src/compiler.h Mon Jun 30 13:25:46 2014 UTC
+++ /branches/bleeding_edge/src/compiler.h Tue Jul 8 14:13:50 2014 UTC
@@ -76,10 +76,12 @@
ASSERT(!is_lazy());
flags_ |= IsEval::encode(true);
}
+
void MarkAsGlobal() {
ASSERT(!is_lazy());
flags_ |= IsGlobal::encode(true);
}
+
void set_parameter_count(int parameter_count) {
ASSERT(IsStub());
parameter_count_ = parameter_count;
@@ -88,13 +90,16 @@
void set_this_has_uses(bool has_no_uses) {
this_has_uses_ = has_no_uses;
}
+
bool this_has_uses() {
return this_has_uses_;
}
+
void SetStrictMode(StrictMode strict_mode) {
ASSERT(this->strict_mode() == SLOPPY || this->strict_mode() ==
strict_mode);
flags_ = StrictModeField::update(flags_, strict_mode);
}
+
void MarkAsNative() {
flags_ |= IsNative::encode(true);
}
@@ -154,9 +159,17 @@
bool is_debug() const {
return IsDebug::decode(flags_);
}
+
+ void PrepareForSerializing() {
+ ASSERT(!is_lazy());
+ flags_ |= PrepareForSerializing::encode(true);
+ }
+
+ bool will_serialize() const { return
PrepareForSerializing::decode(flags_); }
bool IsCodePreAgingActive() const {
- return FLAG_optimize_for_size && FLAG_age_code && !is_debug();
+ return FLAG_optimize_for_size && FLAG_age_code && !will_serialize() &&
+ !is_debug();
}
void SetParseRestriction(ParseRestriction restriction) {
@@ -393,6 +406,8 @@
class RequiresFrame: public BitField<bool, 13, 1> {};
// If the function cannot build a frame (for unspecified reasons)
class MustNotHaveEagerFrame: public BitField<bool, 14, 1> {};
+ // If we plan to serialize the compiled code.
+ class PrepareForSerializing : public BitField<bool, 15, 1> {};
unsigned flags_;
=======================================
--- /branches/bleeding_edge/src/full-codegen.cc Tue Jul 8 09:04:08 2014 UTC
+++ /branches/bleeding_edge/src/full-codegen.cc Tue Jul 8 14:13:50 2014 UTC
@@ -301,10 +301,7 @@
CodeGenerator::MakeCodePrologue(info, "full");
const int kInitialBufferSize = 4 * KB;
MacroAssembler masm(info->isolate(), NULL, kInitialBufferSize);
- if (FLAG_serialize_toplevel &&
- info->cached_data_mode() == PRODUCE_CACHED_DATA &&
info->is_global()) {
- masm.enable_serializer();
- }
+ if (info->will_serialize()) masm.enable_serializer();
#ifdef ENABLE_GDB_JIT_INTERFACE
masm.positions_recorder()->StartGDBJITLineInfoRecording();
=======================================
--- /branches/bleeding_edge/test/cctest/cctest.status Tue Jul 8 10:22:26
2014 UTC
+++ /branches/bleeding_edge/test/cctest/cctest.status Tue Jul 8 14:13:50
2014 UTC
@@ -75,9 +75,6 @@
# BUG(3287). (test-cpu-profiler/SampleWhenFrameIsNotSetup)
'test-cpu-profiler/*': [PASS, FLAKY],
- # TODO(yangguo): Temporarily disable code serializer test
- 'test-compiler/SerializeToplevel': [SKIP],
-
############################################################################
# Slow tests.
'test-api/Threading1': [PASS, ['mode == debug', SLOW]],
=======================================
--- /branches/bleeding_edge/test/cctest/test-compiler.cc Tue Jul 8
09:04:08 2014 UTC
+++ /branches/bleeding_edge/test/cctest/test-compiler.cc Tue Jul 8
14:13:50 2014 UTC
@@ -436,6 +436,8 @@
Handle<Object> result =
Execution::Call(isolate, fun, global, 0, NULL).ToHandleChecked();
CHECK_EQ(2, Handle<Smi>::cast(result)->value());
+
+ delete cache;
}
--
--
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.
