Revision: 24433
Author: [email protected]
Date: Tue Oct 7 12:16:28 2014 UTC
Log: Do not save/restore AST id generator.
AST ids only need to be unique, so there is no need to fiddle around with
them.
[email protected]
Review URL: https://codereview.chromium.org/633053002
https://code.google.com/p/v8/source/detail?r=24433
Modified:
/branches/bleeding_edge/src/parser.cc
/branches/bleeding_edge/src/parser.h
/branches/bleeding_edge/src/preparser.cc
/branches/bleeding_edge/src/preparser.h
=======================================
--- /branches/bleeding_edge/src/parser.cc Mon Oct 6 12:56:11 2014 UTC
+++ /branches/bleeding_edge/src/parser.cc Tue Oct 7 12:16:28 2014 UTC
@@ -342,25 +342,6 @@
//
----------------------------------------------------------------------------
// Implementation of Parser
-class ParserTraits::Checkpoint
- : public ParserBase<ParserTraits>::CheckpointBase {
- public:
- explicit Checkpoint(ParserBase<ParserTraits>* parser)
- : CheckpointBase(parser), parser_(parser) {
- saved_ast_node_id_gen_ = *parser_->ast_node_id_gen_;
- }
-
- void Restore() {
- CheckpointBase::Restore();
- *parser_->ast_node_id_gen_ = saved_ast_node_id_gen_;
- }
-
- private:
- ParserBase<ParserTraits>* parser_;
- AstNode::IdGen saved_ast_node_id_gen_;
-};
-
-
bool ParserTraits::IsEvalOrArguments(const AstRawString* identifier) const
{
return identifier == parser_->ast_value_factory()->eval_string() ||
identifier == parser_->ast_value_factory()->arguments_string();
=======================================
--- /branches/bleeding_edge/src/parser.h Wed Oct 1 16:54:42 2014 UTC
+++ /branches/bleeding_edge/src/parser.h Tue Oct 7 12:16:28 2014 UTC
@@ -374,8 +374,6 @@
typedef AstNodeFactory<AstConstructionVisitor> Factory;
};
- class Checkpoint;
-
explicit ParserTraits(Parser* parser) : parser_(parser) {}
// Custom operations executed when FunctionStates are created and
destructed.
=======================================
--- /branches/bleeding_edge/src/preparser.cc Tue Sep 16 22:15:39 2014 UTC
+++ /branches/bleeding_edge/src/preparser.cc Tue Oct 7 12:16:28 2014 UTC
@@ -32,13 +32,6 @@
namespace v8 {
namespace internal {
-class PreParserTraits::Checkpoint
- : public ParserBase<PreParserTraits>::CheckpointBase {
- public:
- explicit Checkpoint(ParserBase<PreParserTraits>* parser)
- : ParserBase<PreParserTraits>::CheckpointBase(parser) {}
-};
-
void PreParserTraits::ReportMessageAt(Scanner::Location location,
const char* message,
const char* arg,
=======================================
--- /branches/bleeding_edge/src/preparser.h Tue Oct 7 09:20:08 2014 UTC
+++ /branches/bleeding_edge/src/preparser.h Tue Oct 7 12:16:28 2014 UTC
@@ -123,8 +123,6 @@
}
protected:
- friend class Traits::Checkpoint;
-
enum AllowEvalOrArgumentsAsIdentifier {
kAllowEvalOrArguments,
kDontAllowEvalOrArguments
@@ -135,7 +133,7 @@
PARSE_EAGERLY
};
- class CheckpointBase;
+ class Checkpoint;
class ObjectLiteralChecker;
//
---------------------------------------------------------------------------
@@ -235,16 +233,16 @@
typename Traits::Type::Factory factory_;
friend class ParserTraits;
- friend class CheckpointBase;
+ friend class Checkpoint;
};
// Annoyingly, arrow functions first parse as comma expressions, then
when we
// see the => we have to go back and reinterpret the arguments as being
formal
// parameters. To do so we need to reset some of the parser state back
to
// what it was before the arguments were first seen.
- class CheckpointBase BASE_EMBEDDED {
+ class Checkpoint BASE_EMBEDDED {
public:
- explicit CheckpointBase(ParserBase* parser) {
+ explicit Checkpoint(ParserBase* parser) {
function_state_ = parser->function_state_;
next_materialized_literal_index_ =
function_state_->next_materialized_literal_index_;
@@ -1149,8 +1147,6 @@
typedef PreParserFactory Factory;
};
- class Checkpoint;
-
explicit PreParserTraits(PreParser* pre_parser) :
pre_parser_(pre_parser) {}
// Custom operations executed when FunctionStates are created and
@@ -2164,7 +2160,7 @@
}
if (fni_ != NULL) fni_->Enter();
- typename Traits::Checkpoint checkpoint(this);
+ ParserBase<Traits>::Checkpoint checkpoint(this);
ExpressionT expression =
this->ParseConditionalExpression(accept_IN, CHECK_OK);
--
--
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.
