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

Side by Side Diff: runtime/vm/flow_graph_builder.cc

Issue 144343002: Rename kDummyTokenIndex to kNoSourcePos (Closed) Base URL: http://dart.googlecode.com/svn/branches/bleeding_edge/dart/
Patch Set: Created 6 years, 11 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 | « runtime/vm/debugger_api_impl.cc ('k') | runtime/vm/flow_graph_compiler.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 (c) 2012, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2012, the Dart project authors. Please see the AUTHORS file
2 // for details. All rights reserved. Use of this source code is governed by a 2 // for details. All rights reserved. Use of this source code is governed by a
3 // BSD-style license that can be found in the LICENSE file. 3 // BSD-style license that can be found in the LICENSE file.
4 4
5 #include "vm/flow_graph_builder.h" 5 #include "vm/flow_graph_builder.h"
6 6
7 #include "lib/invocation_mirror.h" 7 #include "lib/invocation_mirror.h"
8 #include "vm/ast_printer.h" 8 #include "vm/ast_printer.h"
9 #include "vm/bit_vector.h" 9 #include "vm/bit_vector.h"
10 #include "vm/class_finalizer.h" 10 #include "vm/class_finalizer.h"
(...skipping 952 matching lines...) Expand 10 before | Expand all | Expand 10 after
963 return; 963 return;
964 } 964 }
965 } 965 }
966 966
967 // Call to stub that checks whether the debugger is in single 967 // Call to stub that checks whether the debugger is in single
968 // step mode. This call must happen before the contexts are 968 // step mode. This call must happen before the contexts are
969 // unchained so that captured variables can be inspected. 969 // unchained so that captured variables can be inspected.
970 // No debugger check is done in native functions or for return 970 // No debugger check is done in native functions or for return
971 // statements for which there is no associated source position. 971 // statements for which there is no associated source position.
972 const Function& function = owner()->parsed_function()->function(); 972 const Function& function = owner()->parsed_function()->function();
973 if ((node->token_pos() != Scanner::kDummyTokenIndex) && 973 if ((node->token_pos() != Scanner::kNoSourcePos) &&
974 !function.is_native()) { 974 !function.is_native()) {
975 AddInstruction(new DebugStepCheckInstr(node->token_pos(), 975 AddInstruction(new DebugStepCheckInstr(node->token_pos(),
976 PcDescriptors::kReturn)); 976 PcDescriptors::kReturn));
977 } 977 }
978 978
979 Value* return_value = for_value.value(); 979 Value* return_value = for_value.value();
980 if (FLAG_enable_type_checks) { 980 if (FLAG_enable_type_checks) {
981 const bool is_implicit_dynamic_getter = 981 const bool is_implicit_dynamic_getter =
982 (!function.is_static() && 982 (!function.is_static() &&
983 ((function.kind() == RawFunction::kImplicitGetter) || 983 ((function.kind() == RawFunction::kImplicitGetter) ||
(...skipping 3070 matching lines...) Expand 10 before | Expand all | Expand 10 after
4054 LanguageError::kError, 4054 LanguageError::kError,
4055 Heap::kNew, 4055 Heap::kNew,
4056 "FlowGraphBuilder Bailout: %s %s", 4056 "FlowGraphBuilder Bailout: %s %s",
4057 String::Handle(function.name()).ToCString(), 4057 String::Handle(function.name()).ToCString(),
4058 reason)); 4058 reason));
4059 Isolate::Current()->long_jump_base()->Jump(1, error); 4059 Isolate::Current()->long_jump_base()->Jump(1, error);
4060 } 4060 }
4061 4061
4062 4062
4063 } // namespace dart 4063 } // namespace dart
OLDNEW
« no previous file with comments | « runtime/vm/debugger_api_impl.cc ('k') | runtime/vm/flow_graph_compiler.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698