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/runtime/runtime-function.cc

Issue 1762323002: [compiler] Move ClearExceptionFlag into Compiler. (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: More places. Created 4 years, 9 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/runtime/runtime-compiler.cc ('k') | src/runtime/runtime-test.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 2014 the V8 project authors. All rights reserved. 1 // Copyright 2014 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/runtime/runtime-utils.h" 5 #include "src/runtime/runtime-utils.h"
6 6
7 #include "src/accessors.h" 7 #include "src/accessors.h"
8 #include "src/arguments.h" 8 #include "src/arguments.h"
9 #include "src/compiler.h" 9 #include "src/compiler.h"
10 #include "src/frames-inl.h" 10 #include "src/frames-inl.h"
(...skipping 135 matching lines...) Expand 10 before | Expand all | Expand 10 after
146 RUNTIME_FUNCTION(Runtime_SetCode) { 146 RUNTIME_FUNCTION(Runtime_SetCode) {
147 HandleScope scope(isolate); 147 HandleScope scope(isolate);
148 DCHECK(args.length() == 2); 148 DCHECK(args.length() == 2);
149 149
150 CONVERT_ARG_HANDLE_CHECKED(JSFunction, target, 0); 150 CONVERT_ARG_HANDLE_CHECKED(JSFunction, target, 0);
151 CONVERT_ARG_HANDLE_CHECKED(JSFunction, source, 1); 151 CONVERT_ARG_HANDLE_CHECKED(JSFunction, source, 1);
152 152
153 Handle<SharedFunctionInfo> target_shared(target->shared()); 153 Handle<SharedFunctionInfo> target_shared(target->shared());
154 Handle<SharedFunctionInfo> source_shared(source->shared()); 154 Handle<SharedFunctionInfo> source_shared(source->shared());
155 155
156 if (!Compiler::Compile(source, KEEP_EXCEPTION)) { 156 if (!Compiler::Compile(source, Compiler::KEEP_EXCEPTION)) {
157 return isolate->heap()->exception(); 157 return isolate->heap()->exception();
158 } 158 }
159 159
160 // Mark both, the source and the target, as un-flushable because the 160 // Mark both, the source and the target, as un-flushable because the
161 // shared unoptimized code makes them impossible to enqueue in a list. 161 // shared unoptimized code makes them impossible to enqueue in a list.
162 DCHECK(target_shared->code()->gc_metadata() == NULL); 162 DCHECK(target_shared->code()->gc_metadata() == NULL);
163 DCHECK(source_shared->code()->gc_metadata() == NULL); 163 DCHECK(source_shared->code()->gc_metadata() == NULL);
164 target_shared->set_dont_flush(true); 164 target_shared->set_dont_flush(true);
165 source_shared->set_dont_flush(true); 165 source_shared->set_dont_flush(true);
166 166
(...skipping 133 matching lines...) Expand 10 before | Expand all | Expand 10 after
300 DCHECK_EQ(1, args.length()); 300 DCHECK_EQ(1, args.length());
301 CONVERT_ARG_HANDLE_CHECKED(JSReceiver, function, 0); 301 CONVERT_ARG_HANDLE_CHECKED(JSReceiver, function, 0);
302 return function->IsJSBoundFunction() 302 return function->IsJSBoundFunction()
303 ? *JSBoundFunction::ToString( 303 ? *JSBoundFunction::ToString(
304 Handle<JSBoundFunction>::cast(function)) 304 Handle<JSBoundFunction>::cast(function))
305 : *JSFunction::ToString(Handle<JSFunction>::cast(function)); 305 : *JSFunction::ToString(Handle<JSFunction>::cast(function));
306 } 306 }
307 307
308 } // namespace internal 308 } // namespace internal
309 } // namespace v8 309 } // namespace v8
OLDNEW
« no previous file with comments | « src/runtime/runtime-compiler.cc ('k') | src/runtime/runtime-test.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698