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

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

Issue 10513008: Fix super call invocation (issue 2028). (Closed) Base URL: http://dart.googlecode.com/svn/branches/bleeding_edge/dart/
Patch Set: Created 8 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 | Annotate | Revision Log
« no previous file with comments | « no previous file | runtime/vm/parser.h » ('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/globals.h" // Needed here to get TARGET_ARCH_IA32. 5 #include "vm/globals.h" // Needed here to get TARGET_ARCH_IA32.
6 #if defined(TARGET_ARCH_IA32) 6 #if defined(TARGET_ARCH_IA32)
7 7
8 #include "vm/code_generator.h" 8 #include "vm/code_generator.h"
9 9
10 #include "lib/error.h" 10 #include "lib/error.h"
(...skipping 525 matching lines...) Expand 10 before | Expand all | Expand 10 after
536 delete[] opt_param; 536 delete[] opt_param;
537 delete[] opt_param_position; 537 delete[] opt_param_position;
538 // Check that EDI now points to the null terminator in the array descriptor. 538 // Check that EDI now points to the null terminator in the array descriptor.
539 Label all_arguments_processed; 539 Label all_arguments_processed;
540 __ cmpl(Address(EDI, 0), raw_null); 540 __ cmpl(Address(EDI, 0), raw_null);
541 __ j(EQUAL, &all_arguments_processed, Assembler::kNearJump); 541 __ j(EQUAL, &all_arguments_processed, Assembler::kNearJump);
542 542
543 __ Bind(&wrong_num_arguments); 543 __ Bind(&wrong_num_arguments);
544 if (locals_space_size() != 0) { 544 if (locals_space_size() != 0) {
545 // We need to unwind the space we reserved for locals and copied 545 // We need to unwind the space we reserved for locals and copied
546 // parmeters. The NoSuchMethodFunction stub does not expect to 546 // parameters. The NoSuchMethodFunction stub does not expect to
547 // see that area on the stack. 547 // see that area on the stack.
548 __ addl(ESP, Immediate(locals_space_size())); 548 __ addl(ESP, Immediate(locals_space_size()));
549 } 549 }
550 if (function.IsClosureFunction()) { 550 if (function.IsClosureFunction()) {
551 GenerateCallRuntime(AstNode::kNoId, 551 GenerateCallRuntime(AstNode::kNoId,
552 0, 552 0,
553 kClosureArgumentMismatchRuntimeEntry); 553 kClosureArgumentMismatchRuntimeEntry);
554 } else { 554 } else {
555 // Invoke noSuchMethod function. 555 // Invoke noSuchMethod function.
556 const int kNumArgsChecked = 1; 556 const int kNumArgsChecked = 1;
(...skipping 2386 matching lines...) Expand 10 before | Expand all | Expand 10 after
2943 const Error& error = Error::Handle( 2943 const Error& error = Error::Handle(
2944 Parser::FormatError(script, token_index, "Error", format, args)); 2944 Parser::FormatError(script, token_index, "Error", format, args));
2945 va_end(args); 2945 va_end(args);
2946 Isolate::Current()->long_jump_base()->Jump(1, error); 2946 Isolate::Current()->long_jump_base()->Jump(1, error);
2947 UNREACHABLE(); 2947 UNREACHABLE();
2948 } 2948 }
2949 2949
2950 } // namespace dart 2950 } // namespace dart
2951 2951
2952 #endif // defined TARGET_ARCH_IA32 2952 #endif // defined TARGET_ARCH_IA32
OLDNEW
« no previous file with comments | « no previous file | runtime/vm/parser.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698