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

Side by Side Diff: src/hydrogen.cc

Issue 137403009: Adding a type vector to replace type cells. (Closed) Base URL: https://v8.googlecode.com/svn/branches/bleeding_edge
Patch Set: Seperate file for feedback slot allocation. Created 6 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 | Annotate | Revision Log
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
(...skipping 40 matching lines...) Expand 10 before | Expand all | Expand 10 after
51 #include "hydrogen-minus-zero.h" 51 #include "hydrogen-minus-zero.h"
52 #include "hydrogen-osr.h" 52 #include "hydrogen-osr.h"
53 #include "hydrogen-range-analysis.h" 53 #include "hydrogen-range-analysis.h"
54 #include "hydrogen-redundant-phi.h" 54 #include "hydrogen-redundant-phi.h"
55 #include "hydrogen-removable-simulates.h" 55 #include "hydrogen-removable-simulates.h"
56 #include "hydrogen-representation-changes.h" 56 #include "hydrogen-representation-changes.h"
57 #include "hydrogen-sce.h" 57 #include "hydrogen-sce.h"
58 #include "hydrogen-uint32-analysis.h" 58 #include "hydrogen-uint32-analysis.h"
59 #include "lithium-allocator.h" 59 #include "lithium-allocator.h"
60 #include "parser.h" 60 #include "parser.h"
61 #include "feedbackslots.h"
61 #include "runtime.h" 62 #include "runtime.h"
62 #include "scopeinfo.h" 63 #include "scopeinfo.h"
63 #include "scopes.h" 64 #include "scopes.h"
64 #include "stub-cache.h" 65 #include "stub-cache.h"
65 #include "typing.h" 66 #include "typing.h"
66 67
67 #if V8_TARGET_ARCH_IA32 68 #if V8_TARGET_ARCH_IA32
68 #include "ia32/lithium-codegen-ia32.h" 69 #include "ia32/lithium-codegen-ia32.h"
69 #elif V8_TARGET_ARCH_X64 70 #elif V8_TARGET_ARCH_X64
70 #include "x64/lithium-codegen-x64.h" 71 #include "x64/lithium-codegen-x64.h"
(...skipping 7088 matching lines...) Expand 10 before | Expand all | Expand 10 after
7159 // We don't want to add more than a certain number of nodes from inlining. 7160 // We don't want to add more than a certain number of nodes from inlining.
7160 if (inlined_count_ > Min(FLAG_max_inlined_nodes_cumulative, 7161 if (inlined_count_ > Min(FLAG_max_inlined_nodes_cumulative,
7161 kUnlimitedMaxInlinedNodesCumulative)) { 7162 kUnlimitedMaxInlinedNodesCumulative)) {
7162 TraceInline(target, caller, "cumulative AST node limit reached"); 7163 TraceInline(target, caller, "cumulative AST node limit reached");
7163 return false; 7164 return false;
7164 } 7165 }
7165 7166
7166 // Parse and allocate variables. 7167 // Parse and allocate variables.
7167 CompilationInfo target_info(target, zone()); 7168 CompilationInfo target_info(target, zone());
7168 Handle<SharedFunctionInfo> target_shared(target->shared()); 7169 Handle<SharedFunctionInfo> target_shared(target->shared());
7169 if (!Parser::Parse(&target_info) || !Scope::Analyze(&target_info)) { 7170 if (!Parser::Parse(&target_info) || !Scope::Analyze(&target_info) ||
7171 !FeedbackSlotAllocator::Run(&target_info)) {
7170 if (target_info.isolate()->has_pending_exception()) { 7172 if (target_info.isolate()->has_pending_exception()) {
7171 // Parse or scope error, never optimize this function. 7173 // Parse or scope error, never optimize this function.
7172 SetStackOverflow(); 7174 SetStackOverflow();
7173 target_shared->DisableOptimization(kParseScopeError); 7175 target_shared->DisableOptimization(kParseScopeError);
7174 } 7176 }
7175 TraceInline(target, caller, "parse failure"); 7177 TraceInline(target, caller, "parse failure");
7176 return false; 7178 return false;
7177 } 7179 }
7178 7180
7179 if (target_info.scope()->num_heap_slots() > 0) { 7181 if (target_info.scope()->num_heap_slots() > 0) {
(...skipping 3903 matching lines...) Expand 10 before | Expand all | Expand 10 after
11083 if (ShouldProduceTraceOutput()) { 11085 if (ShouldProduceTraceOutput()) {
11084 isolate()->GetHTracer()->TraceHydrogen(name(), graph_); 11086 isolate()->GetHTracer()->TraceHydrogen(name(), graph_);
11085 } 11087 }
11086 11088
11087 #ifdef DEBUG 11089 #ifdef DEBUG
11088 graph_->Verify(false); // No full verify. 11090 graph_->Verify(false); // No full verify.
11089 #endif 11091 #endif
11090 } 11092 }
11091 11093
11092 } } // namespace v8::internal 11094 } } // namespace v8::internal
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698