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

Side by Side Diff: src/debug.cc

Issue 1211803002: Reland 2 "Keep a canonical list of shared function infos." (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: add test case Created 5 years, 6 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/ast-graph-builder.cc ('k') | src/factory.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 2012 the V8 project authors. All rights reserved. 1 // Copyright 2012 the V8 project authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "src/v8.h" 5 #include "src/v8.h"
6 6
7 #include "src/api.h" 7 #include "src/api.h"
8 #include "src/arguments.h" 8 #include "src/arguments.h"
9 #include "src/bootstrapper.h" 9 #include "src/bootstrapper.h"
10 #include "src/code-stubs.h" 10 #include "src/code-stubs.h"
(...skipping 2061 matching lines...) Expand 10 before | Expand all | Expand 10 after
2072 2072
2073 // There will be at least one break point when we are done. 2073 // There will be at least one break point when we are done.
2074 has_break_points_ = true; 2074 has_break_points_ = true;
2075 2075
2076 // If the candidate found is compiled we are done. 2076 // If the candidate found is compiled we are done.
2077 done = target->is_compiled(); 2077 done = target->is_compiled();
2078 if (!done) { 2078 if (!done) {
2079 // If the candidate is not compiled, compile it to reveal any inner 2079 // If the candidate is not compiled, compile it to reveal any inner
2080 // functions which might contain the requested source position. This 2080 // functions which might contain the requested source position. This
2081 // will compile all inner functions that cannot be compiled without a 2081 // will compile all inner functions that cannot be compiled without a
2082 // context, because Compiler::BuildFunctionInfo checks whether the 2082 // context, because Compiler::GetSharedFunctionInfo checks whether the
2083 // debugger is active. 2083 // debugger is active.
2084 MaybeHandle<Code> maybe_result = target_function.is_null() 2084 MaybeHandle<Code> maybe_result = target_function.is_null()
2085 ? Compiler::GetUnoptimizedCode(target) 2085 ? Compiler::GetUnoptimizedCode(target)
2086 : Compiler::GetUnoptimizedCode(target_function); 2086 : Compiler::GetUnoptimizedCode(target_function);
2087 if (maybe_result.is_null()) return isolate_->factory()->undefined_value(); 2087 if (maybe_result.is_null()) return isolate_->factory()->undefined_value();
2088 } 2088 }
2089 } // End while loop. 2089 } // End while loop.
2090 2090
2091 // JSFunctions from the same literal may not have the same shared function 2091 // JSFunctions from the same literal may not have the same shared function
2092 // info. Find those JSFunctions and deduplicate the shared function info. 2092 // info. Find those JSFunctions and deduplicate the shared function info.
(...skipping 1294 matching lines...) Expand 10 before | Expand all | Expand 10 after
3387 } 3387 }
3388 3388
3389 3389
3390 void LockingCommandMessageQueue::Clear() { 3390 void LockingCommandMessageQueue::Clear() {
3391 base::LockGuard<base::Mutex> lock_guard(&mutex_); 3391 base::LockGuard<base::Mutex> lock_guard(&mutex_);
3392 queue_.Clear(); 3392 queue_.Clear();
3393 } 3393 }
3394 3394
3395 } // namespace internal 3395 } // namespace internal
3396 } // namespace v8 3396 } // namespace v8
OLDNEW
« no previous file with comments | « src/compiler/ast-graph-builder.cc ('k') | src/factory.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698