Revert "TransitionAndStoreStub bailout needs to transition (and store)."

This reverts commit r17216 breaking fast/js/cross-frame-bad-time.html test.

R=mvstanton@chromium.org
TEST=webkit:fast/js/cross-frame-bad-time.html

Review URL: https://codereview.chromium.org/27516002

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@17241 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
parent 24a7e063
...@@ -2257,14 +2257,9 @@ RUNTIME_FUNCTION(MaybeObject*, ElementsTransitionAndStoreIC_Miss) { ...@@ -2257,14 +2257,9 @@ RUNTIME_FUNCTION(MaybeObject*, ElementsTransitionAndStoreIC_Miss) {
ASSERT(args.length() == 4); ASSERT(args.length() == 4);
KeyedStoreIC ic(IC::EXTRA_CALL_FRAME, isolate); KeyedStoreIC ic(IC::EXTRA_CALL_FRAME, isolate);
Handle<Object> value = args.at<Object>(0); Handle<Object> value = args.at<Object>(0);
Handle<Map> map = args.at<Map>(1);
Handle<Object> key = args.at<Object>(2); Handle<Object> key = args.at<Object>(2);
Handle<Object> object = args.at<Object>(3); Handle<Object> object = args.at<Object>(3);
StrictModeFlag strict_mode = ic.strict_mode(); StrictModeFlag strict_mode = ic.strict_mode();
if (object->IsJSObject()) {
JSObject::TransitionElementsKind(Handle<JSObject>::cast(object),
map->elements_kind());
}
return Runtime::SetObjectProperty(isolate, return Runtime::SetObjectProperty(isolate,
object, object,
key, key,
......
...@@ -148,12 +148,8 @@ if (support_smi_only_arrays) { ...@@ -148,12 +148,8 @@ if (support_smi_only_arrays) {
assertKind(elements_kind.fast_double, obj); assertKind(elements_kind.fast_double, obj);
obj = fastliteralcase([3, 6, 2], 1.5); obj = fastliteralcase([3, 6, 2], 1.5);
assertKind(elements_kind.fast_double, obj); assertKind(elements_kind.fast_double, obj);
// Note: thanks to pessimistic transition store stubs, we'll attempt
// to transition to the most general elements kind seen at a particular
// store site. So, the elements kind will be double.
obj = fastliteralcase([2, 6, 3], 2); obj = fastliteralcase([2, 6, 3], 2);
assertKind(elements_kind.fast_double, obj); assertKind(elements_kind.fast_smi_only, obj);
} }
// Verify that we will not pretransition the double->fast path. // Verify that we will not pretransition the double->fast path.
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment