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

Side by Side Diff: src/runtime.js

Issue 151018: Optimize %ClassOf() on IA-32 and use it instead of the... (Closed) Base URL: http://v8.googlecode.com/svn/branches/bleeding_edge/
Patch Set: Created 11 years, 5 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 | « src/runtime.cc ('k') | src/string.js » ('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 2006-2008 the V8 project authors. All rights reserved. 1 // Copyright 2006-2008 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 376 matching lines...) Expand 10 before | Expand all | Expand 10 after
387 var parameters = %NewArguments(delegate); 387 var parameters = %NewArguments(delegate);
388 return delegate.apply(callee, parameters); 388 return delegate.apply(callee, parameters);
389 } 389 }
390 390
391 391
392 function APPLY_PREPARE(args) { 392 function APPLY_PREPARE(args) {
393 var length; 393 var length;
394 // First check whether length is a positive Smi and args is an 394 // First check whether length is a positive Smi and args is an
395 // array. This is the fast case. If this fails, we do the slow case 395 // array. This is the fast case. If this fails, we do the slow case
396 // that takes care of more eventualities. 396 // that takes care of more eventualities.
397 if (%_IsArray(args)) { 397 if (IS_ARRAY(args)) {
398 length = args.length; 398 length = args.length;
399 if (%_IsSmi(length) && length >= 0 && length < 0x800000 && IS_FUNCTION(this) ) { 399 if (%_IsSmi(length) && length >= 0 && length < 0x800000 && IS_FUNCTION(this) ) {
400 return length; 400 return length;
401 } 401 }
402 } 402 }
403 403
404 length = (args == null) ? 0 : %ToUint32(args.length); 404 length = (args == null) ? 0 : %ToUint32(args.length);
405 405
406 // We can handle any number of apply arguments if the stack is 406 // We can handle any number of apply arguments if the stack is
407 // big enough, but sanity check the value to avoid overflow when 407 // big enough, but sanity check the value to avoid overflow when
408 // multiplying with pointer size. 408 // multiplying with pointer size.
409 if (length > 0x800000) { 409 if (length > 0x800000) {
410 throw %MakeRangeError('apply_overflow', [length]); 410 throw %MakeRangeError('apply_overflow', [length]);
411 } 411 }
412 412
413 if (!IS_FUNCTION(this)) { 413 if (!IS_FUNCTION(this)) {
414 throw %MakeTypeError('apply_non_function', [ %ToString(this), typeof this ]) ; 414 throw %MakeTypeError('apply_non_function', [ %ToString(this), typeof this ]) ;
415 } 415 }
416 416
417 // Make sure the arguments list has the right type. 417 // Make sure the arguments list has the right type.
418 if (args != null && 418 if (args != null && !IS_ARRAY(args) && !IS_ARGUMENTS(args)) {
419 !%HasArrayClass(args) &&
420 !%HasArgumentsClass(args)) {
421 throw %MakeTypeError('apply_wrong_args', []); 419 throw %MakeTypeError('apply_wrong_args', []);
422 } 420 }
423 421
424 // Return the length which is the number of arguments to copy to the 422 // Return the length which is the number of arguments to copy to the
425 // stack. It is guaranteed to be a small integer at this point. 423 // stack. It is guaranteed to be a small integer at this point.
426 return length; 424 return length;
427 } 425 }
428 426
429 427
430 function APPLY_OVERFLOW(length) { 428 function APPLY_OVERFLOW(length) {
(...skipping 148 matching lines...) Expand 10 before | Expand all | Expand 10 after
579 throw %MakeTypeError('cannot_convert_to_primitive', []); 577 throw %MakeTypeError('cannot_convert_to_primitive', []);
580 } 578 }
581 579
582 580
583 // NOTE: Setting the prototype for Array must take place as early as 581 // NOTE: Setting the prototype for Array must take place as early as
584 // possible due to code generation for array literals. When 582 // possible due to code generation for array literals. When
585 // generating code for a array literal a boilerplate array is created 583 // generating code for a array literal a boilerplate array is created
586 // that is cloned when running the code. It is essiential that the 584 // that is cloned when running the code. It is essiential that the
587 // boilerplate gets the right prototype. 585 // boilerplate gets the right prototype.
588 %FunctionSetPrototype($Array, new $Array(0)); 586 %FunctionSetPrototype($Array, new $Array(0));
OLDNEW
« no previous file with comments | « src/runtime.cc ('k') | src/string.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698