Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(161)

Side by Side Diff: src/data-flow.h

Issue 553149: Implement simple fast-path code for functions containing this property stores and global variables. (Closed)
Patch Set: Incorporated codereview comments. Created 10 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « src/compiler.cc ('k') | src/data-flow.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2010 the V8 project authors. All rights reserved. 1 // Copyright 2010 the V8 project authors. All rights reserved.
2 // Redistribution and use in source and binary forms, with or without 2 // Redistribution and use in source and binary forms, with or without
3 // modification, are permitted provided that the following conditions are 3 // modification, are permitted provided that the following conditions are
4 // met: 4 // met:
5 // 5 //
6 // * Redistributions of source code must retain the above copyright 6 // * Redistributions of source code must retain the above copyright
7 // notice, this list of conditions and the following disclaimer. 7 // notice, this list of conditions and the following disclaimer.
8 // * Redistributions in binary form must reproduce the above 8 // * Redistributions in binary form must reproduce the above
9 // copyright notice, this list of conditions and the following 9 // copyright notice, this list of conditions and the following
10 // disclaimer in the documentation and/or other materials provided 10 // disclaimer in the documentation and/or other materials provided
(...skipping 20 matching lines...) Expand all
31 #include "ast.h" 31 #include "ast.h"
32 #include "scopes.h" 32 #include "scopes.h"
33 33
34 namespace v8 { 34 namespace v8 {
35 namespace internal { 35 namespace internal {
36 36
37 // This class is used to number all expressions in the AST according to 37 // This class is used to number all expressions in the AST according to
38 // their evaluation order (post-order left-to-right traversal). 38 // their evaluation order (post-order left-to-right traversal).
39 class AstLabeler: public AstVisitor { 39 class AstLabeler: public AstVisitor {
40 public: 40 public:
41 AstLabeler() : next_number_(0) {} 41 AstLabeler() : next_number_(0), has_this_properties_(false) {}
42 42
43 void Label(FunctionLiteral* fun); 43 void Label(FunctionLiteral* fun);
44 44
45 bool has_this_properties() { return has_this_properties_; }
46
45 private: 47 private:
46 void VisitDeclarations(ZoneList<Declaration*>* decls); 48 void VisitDeclarations(ZoneList<Declaration*>* decls);
47 void VisitStatements(ZoneList<Statement*>* stmts); 49 void VisitStatements(ZoneList<Statement*>* stmts);
48 50
49 // AST node visit functions. 51 // AST node visit functions.
50 #define DECLARE_VISIT(type) virtual void Visit##type(type* node); 52 #define DECLARE_VISIT(type) virtual void Visit##type(type* node);
51 AST_NODE_LIST(DECLARE_VISIT) 53 AST_NODE_LIST(DECLARE_VISIT)
52 #undef DECLARE_VISIT 54 #undef DECLARE_VISIT
53 55
54 // Traversal number for labelling AST nodes. 56 // Traversal number for labelling AST nodes.
55 int next_number_; 57 int next_number_;
56 58
59 bool has_this_properties_;
60
57 DISALLOW_COPY_AND_ASSIGN(AstLabeler); 61 DISALLOW_COPY_AND_ASSIGN(AstLabeler);
58 }; 62 };
59 63
60 64
61 } } // namespace v8::internal 65 } } // namespace v8::internal
62 66
63 #endif // V8_DATAFLOW_H_ 67 #endif // V8_DATAFLOW_H_
OLDNEW
« no previous file with comments | « src/compiler.cc ('k') | src/data-flow.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698