Avoid virtual call for array length in the uncached case.

woess · woess · commit 5a912d6a8986 · 2024-01-30T15:19:06.000+01:00
diff --git a/graal-js/src/com.oracle.truffle.js/src/com/oracle/truffle/js/nodes/interop/ArrayElementInfoNode.java b/graal-js/src/com.oracle.truffle.js/src/com/oracle/truffle/js/nodes/interop/ArrayElementInfoNode.java
@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2020, 2020, Oracle and/or its affiliates. All rights reserved.
+ * Copyright (c) 2020, 2024, Oracle and/or its affiliates. All rights reserved.
  * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
  *
  * The Universal Permissive License (UPL), Version 1.0
@@ -49,6 +49,7 @@
 import com.oracle.truffle.js.nodes.JavaScriptBaseNode;
 import com.oracle.truffle.js.runtime.JSRuntime;
 import com.oracle.truffle.js.runtime.array.ScriptArray;
+import com.oracle.truffle.js.runtime.builtins.JSAbstractArray;
 import com.oracle.truffle.js.runtime.builtins.JSArrayBase;
 
 /**
@@ -99,7 +100,7 @@ static TriState doCached(JSArrayBase target, long index, int query,
                 return TriState.UNDEFINED;
             }
         }
-        if (index >= 0 && index < arrayType.length(target)) {
+        if (index >= 0 && index < JSAbstractArray.arrayGetLength(target)) {
             if ((query & READABLE) != 0) {
                 return TriState.TRUE;
             }

Original file line number	Diff line number	Diff line change
`@@ -1,5 +1,5 @@`
`1`	`1`	`/*`
`2`		`- * Copyright (c) 2020, 2020, Oracle and/or its affiliates. All rights reserved.`
	`2`	`+ * Copyright (c) 2020, 2024, Oracle and/or its affiliates. All rights reserved.`
`3`	`3`	`* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.`
`4`	`4`	`*`
`5`	`5`	`* The Universal Permissive License (UPL), Version 1.0`
`@@ -49,6 +49,7 @@`
`49`	`49`	`import com.oracle.truffle.js.nodes.JavaScriptBaseNode;`
`50`	`50`	`import com.oracle.truffle.js.runtime.JSRuntime;`
`51`	`51`	`import com.oracle.truffle.js.runtime.array.ScriptArray;`
	`52`	`+import com.oracle.truffle.js.runtime.builtins.JSAbstractArray;`
`52`	`53`	`import com.oracle.truffle.js.runtime.builtins.JSArrayBase;`
`53`	`54`
`54`	`55`	`/**`
`@@ -99,7 +100,7 @@ static TriState doCached(JSArrayBase target, long index, int query,`
`99`	`100`	`return TriState.UNDEFINED;`
`100`	`101`	`}`
`101`	`102`	`}`
`102`		`- if (index >= 0 && index < arrayType.length(target)) {`
	`103`	`+ if (index >= 0 && index < JSAbstractArray.arrayGetLength(target)) {`
`103`	`104`	`if ((query & READABLE) != 0) {`
`104`	`105`	`return TriState.TRUE;`
`105`	`106`	`}`