61ce45c9e2
With bytecode flushing and lazy feedback allocation, we need to call %PrepareForOptimization before we call %OptimizeFunctionOnNextCall, ideally after declaring the function. Bug: v8:8801, v8:8394, v8:9183 Change-Id: I6bf119e726426df8527d97546b6ce806112c894d Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1643167 Auto-Submit: Mathias Bynens <mathias@chromium.org> Reviewed-by: Mythri Alle <mythria@chromium.org> Commit-Queue: Mathias Bynens <mathias@chromium.org> Cr-Commit-Position: refs/heads/master@{#61988}
25 lines
707 B
JavaScript
25 lines
707 B
JavaScript
// Copyright 2014 the V8 project authors. All rights reserved.
|
|
// Use of this source code is governed by a BSD-style license that can be
|
|
// found in the LICENSE file.
|
|
|
|
// Flags: --allow-natives-syntax
|
|
|
|
function push_wrapper(array, value) {
|
|
array.push(value);
|
|
}
|
|
|
|
%PrepareFunctionForOptimization(push_wrapper);
|
|
|
|
// Test that optimization of Array.push() for non-Arrays works correctly.
|
|
var object = { x : 8, length: 3 };
|
|
object[18] = 5;
|
|
object.__proto__ = Array.prototype;
|
|
push_wrapper(object, 1);
|
|
push_wrapper(object, 1);
|
|
assertEquals(5, object.length);
|
|
%OptimizeFunctionOnNextCall(push_wrapper);
|
|
push_wrapper(object, 1);
|
|
push_wrapper(object, 1);
|
|
assertEquals(8, object.x);
|
|
assertEquals(7, object.length);
|