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

Unified Diff: src/runtime.js

Issue 1295433002: [runtime] Remove useless IN builtin. (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: REBASE again. Remove unused ReplaceWithBuiltinCall. Created 5 years, 3 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « src/hydrogen.cc ('k') | src/runtime/runtime.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/runtime.js
diff --git a/src/runtime.js b/src/runtime.js
index 880b7dea1d84e0fc985f6b51a93f578b776cb474..b2e902288946dbbfa0ef77e359ba38c66a124057 100644
--- a/src/runtime.js
+++ b/src/runtime.js
@@ -424,21 +424,6 @@ function SHR_STRONG(y) {
-----------------------------
*/
-// ECMA-262, section 11.8.7, page 54.
-function IN(x) {
- if (!IS_SPEC_OBJECT(x)) {
- throw %make_type_error(kInvalidInOperatorUse, this, x);
- }
- if (%_IsNonNegativeSmi(this)) {
- if (IS_ARRAY(x) && %_HasFastPackedElements(x)) {
- return this < x.length;
- }
- return %HasElement(x, this);
- }
- return %HasProperty(x, this);
-}
-
-
function CALL_NON_FUNCTION() {
var delegate = %GetFunctionDelegate(this);
return %Apply(delegate, this, arguments, 0, %_ArgumentsLength());
@@ -826,7 +811,6 @@ $toString = ToString;
"div_builtin", DIV,
"div_strong_builtin", DIV_STRONG,
"equals_builtin", EQUALS,
- "in_builtin", IN,
"mod_builtin", MOD,
"mod_strong_builtin", MOD_STRONG,
"mul_builtin", MUL,
« no previous file with comments | « src/hydrogen.cc ('k') | src/runtime/runtime.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698