Revision: 5485
Author: [email protected]
Date: Fri Sep 17 05:55:27 2010
Log: Make preparsing data reusable.
Fix bug in writing symbol ids over 128.
Review URL: http://codereview.chromium.org/3416010
http://code.google.com/p/v8/source/detail?r=5485
Modified:
/branches/bleeding_edge/src/parser.cc
/branches/bleeding_edge/src/parser.h
=======================================
--- /branches/bleeding_edge/src/parser.cc Wed Sep 15 03:54:35 2010
+++ /branches/bleeding_edge/src/parser.cc Fri Sep 17 05:55:27 2010
@@ -1001,7 +1001,7 @@
Vector<Vector<const char> > symbol = symbol_entries_.AddBlock(1,
literal);
entry->key = &symbol[0];
}
- symbol_store_.Add(id - 1);
+ WriteNumber(id - 1);
}
virtual Vector<unsigned> ExtractData() {
@@ -1503,6 +1503,7 @@
source->TryFlatten();
scanner_.Initialize(source, JAVASCRIPT);
ASSERT(target_stack_ == NULL);
+ if (pre_data_ != NULL) pre_data_->Initialize();
// Compute the parsing mode.
mode_ = FLAG_lazy ? PARSE_LAZILY : PARSE_EAGERLY;
@@ -5492,7 +5493,9 @@
void ScriptDataImpl::Initialize() {
+ // Prepares state for use.
if (store_.length() >= kHeaderSize) {
+ function_index_ = kHeaderSize;
int symbol_data_offset = kHeaderSize + store_[kFunctionsSizeOffset];
if (store_.length() > symbol_data_offset) {
symbol_data_ = reinterpret_cast<byte*>(&store_[symbol_data_offset]);
=======================================
--- /branches/bleeding_edge/src/parser.h Wed Sep 15 03:54:35 2010
+++ /branches/bleeding_edge/src/parser.h Fri Sep 17 05:55:27 2010
@@ -101,10 +101,7 @@
public:
explicit ScriptDataImpl(Vector<unsigned> store)
: store_(store),
- function_index_(kHeaderSize),
- owns_store_(true) {
- Initialize();
- }
+ owns_store_(true) { }
// Create an empty ScriptDataImpl that is guaranteed to not satisfy
// a SanityCheck.
@@ -190,10 +187,8 @@
ScriptDataImpl(const char* backing_store, int length)
:
store_(reinterpret_cast<unsigned*>(const_cast<char*>(backing_store)),
length / sizeof(unsigned)),
- function_index_(kHeaderSize),
owns_store_(false) {
ASSERT_EQ(0, reinterpret_cast<intptr_t>(backing_store) %
sizeof(unsigned));
- Initialize();
}
// Read strings written by ParserRecorder::WriteString.
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
