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

Side by Side Diff: src/lithium-codegen.h

Issue 22876009: Improve and simplify removal of unreachable code (Closed) Base URL: https://v8.googlecode.com/svn/branches/bleeding_edge
Patch Set: Address review feedback Created 7 years, 2 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 | Annotate | Revision Log
« no previous file with comments | « src/lithium.cc ('k') | src/lithium-codegen.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 2013 the V8 project authors. All rights reserved. 1 // Copyright 2013 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
11 // with the distribution. 11 // with the distribution.
12 // * Neither the name of Google Inc. nor the names of its 12 // * Neither the name of Google Inc. nor the names of its
13 // contributors may be used to endorse or promote products derived 13 // contributors may be used to endorse or promote products derived
14 // from this software without specific prior written permission. 14 // from this software without specific prior written permission.
15 // 15 //
16 // THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS 16 // THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
17 // "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT 17 // "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
18 // LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR 18 // LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
19 // A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT 19 // A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
20 // OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, 20 // OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
21 // SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT 21 // SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
22 // LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, 22 // LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
23 // DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY 23 // DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
24 // THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT 24 // THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
25 // (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE 25 // (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
26 // OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. 26 // OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
27 27
28 #ifndef V8_TYPING_H_ 28 #ifndef V8_LITHIUM_CODEGEN_H_
29 #define V8_TYPING_H_ 29 #define V8_LITHIUM_CODEGEN_H_
30 30
31 #include "v8.h" 31 #include "v8.h"
32 32
33 #include "allocation.h"
34 #include "ast.h"
35 #include "compiler.h" 33 #include "compiler.h"
36 #include "type-info.h"
37 #include "types.h"
38 #include "effects.h"
39 #include "zone.h"
40 #include "scopes.h"
41 34
42 namespace v8 { 35 namespace v8 {
43 namespace internal { 36 namespace internal {
44 37
38 class LInstruction;
39 class LPlatformChunk;
45 40
46 class AstTyper: public AstVisitor { 41 class LCodeGenBase BASE_EMBEDDED {
47 public: 42 public:
48 static void Run(CompilationInfo* info); 43 LCodeGenBase(LChunk* chunk,
44 MacroAssembler* assembler,
45 CompilationInfo* info);
46 virtual ~LCodeGenBase() {}
49 47
50 void* operator new(size_t size, Zone* zone) { 48 // Simple accessors.
51 return zone->New(static_cast<int>(size)); 49 MacroAssembler* masm() const { return masm_; }
52 } 50 CompilationInfo* info() const { return info_; }
53 void operator delete(void* pointer, Zone* zone) { } 51 Isolate* isolate() const { return info_->isolate(); }
54 void operator delete(void* pointer) { } 52 Factory* factory() const { return isolate()->factory(); }
53 Heap* heap() const { return isolate()->heap(); }
54 Zone* zone() const { return zone_; }
55 LPlatformChunk* chunk() const { return chunk_; }
56 HGraph* graph() const;
55 57
56 DEFINE_AST_VISITOR_SUBCLASS_MEMBERS(); 58 void FPRINTF_CHECKING Comment(const char* format, ...);
57 59
58 private: 60 bool GenerateBody();
59 explicit AstTyper(CompilationInfo* info); 61 virtual void GenerateBodyInstructionPre(LInstruction* instr) {}
62 virtual void GenerateBodyInstructionPost(LInstruction* instr) {}
60 63
61 static const int kNoVar = INT_MIN; 64 virtual void EnsureSpaceForLazyDeopt(int space_needed) = 0;
62 typedef v8::internal::Effects<int, kNoVar> Effects; 65 virtual void RecordAndUpdatePosition(int position) = 0;
63 typedef v8::internal::NestedEffects<int, kNoVar> Store;
64 66
65 CompilationInfo* info_; 67 int GetNextEmittedBlock() const;
66 TypeFeedbackOracle oracle_;
67 Store store_;
68 68
69 TypeFeedbackOracle* oracle() { return &oracle_; } 69 protected:
70 Zone* zone() const { return info_->zone(); } 70 enum Status {
71 UNUSED,
72 GENERATING,
73 DONE,
74 ABORTED
75 };
71 76
72 void NarrowType(Expression* e, Bounds b) { 77 LPlatformChunk* const chunk_;
73 e->set_bounds(Bounds::Both(e->bounds(), b, isolate_)); 78 MacroAssembler* const masm_;
74 } 79 CompilationInfo* const info_;
75 void NarrowLowerType(Expression* e, Handle<Type> t) { 80 Zone* zone_;
76 e->set_bounds(Bounds::NarrowLower(e->bounds(), t, isolate_)); 81 Status status_;
77 } 82 int current_block_;
83 int current_instruction_;
84 const ZoneList<LInstruction*>* instructions_;
85 int last_lazy_deopt_pc_;
78 86
79 Effects EnterEffects() { 87 bool is_unused() const { return status_ == UNUSED; }
80 store_ = store_.Push(); 88 bool is_generating() const { return status_ == GENERATING; }
81 return store_.Top(); 89 bool is_done() const { return status_ == DONE; }
82 } 90 bool is_aborted() const { return status_ == ABORTED; }
83 void ExitEffects() { store_ = store_.Pop(); } 91 };
84 92
85 int variable_index(Variable* var) {
86 return var->IsStackLocal() ? var->index() :
87 var->IsParameter() ? -var->index() : kNoVar;
88 }
89
90 void VisitDeclarations(ZoneList<Declaration*>* declarations);
91 void VisitStatements(ZoneList<Statement*>* statements);
92
93 #define DECLARE_VISIT(type) virtual void Visit##type(type* node);
94 AST_NODE_LIST(DECLARE_VISIT)
95 #undef DECLARE_VISIT
96
97 DISALLOW_COPY_AND_ASSIGN(AstTyper);
98 };
99 93
100 } } // namespace v8::internal 94 } } // namespace v8::internal
101 95
102 #endif // V8_TYPING_H_ 96 #endif // V8_LITHIUM_CODEGEN_H_
OLDNEW
« no previous file with comments | « src/lithium.cc ('k') | src/lithium-codegen.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698