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

Unified Diff: src/code-stubs.cc

Issue 2504403005: [stubs] KeyedStoreGeneric: inline dictionary property stores (Closed)
Patch Set: ready for review Created 4 years, 1 month 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
Index: src/code-stubs.cc
diff --git a/src/code-stubs.cc b/src/code-stubs.cc
index f0f38ebe9376962df326dab42cc1abff0f67958a..18acccb204a65ea90f44b197a49a724891eb7dc4 100644
--- a/src/code-stubs.cc
+++ b/src/code-stubs.cc
@@ -2116,7 +2116,7 @@ void StoreScriptContextFieldStub::GenerateAssembly(
Node* script_context = assembler.LoadScriptContext(context, context_index());
assembler.StoreFixedArrayElement(
- script_context, assembler.IntPtrConstant(slot_index()), value,
+ script_context, assembler.IntPtrConstant(slot_index()), value, 0,
UPDATE_WRITE_BARRIER, CodeStubAssembler::INTPTR_PARAMETERS);
assembler.Return(value);
}
@@ -2621,21 +2621,19 @@ compiler::Node* FastNewFunctionContextStub::Generate(
assembler->SmiFromWord32(length));
// Set up the fixed slots.
- assembler->StoreFixedArrayElement(
- function_context, assembler->Int32Constant(Context::CLOSURE_INDEX),
- function, SKIP_WRITE_BARRIER);
- assembler->StoreFixedArrayElement(
- function_context, assembler->Int32Constant(Context::PREVIOUS_INDEX),
- context, SKIP_WRITE_BARRIER);
- assembler->StoreFixedArrayElement(
- function_context, assembler->Int32Constant(Context::EXTENSION_INDEX),
- assembler->TheHoleConstant(), SKIP_WRITE_BARRIER);
+ assembler->StoreFixedArrayElement(function_context, Context::CLOSURE_INDEX,
Jakob Kummerow 2016/11/21 17:02:58 More drive-by cleanup.
+ function, SKIP_WRITE_BARRIER);
+ assembler->StoreFixedArrayElement(function_context, Context::PREVIOUS_INDEX,
+ context, SKIP_WRITE_BARRIER);
+ assembler->StoreFixedArrayElement(function_context, Context::EXTENSION_INDEX,
+ assembler->TheHoleConstant(),
+ SKIP_WRITE_BARRIER);
// Copy the native context from the previous context.
Node* native_context = assembler->LoadNativeContext(context);
- assembler->StoreFixedArrayElement(
- function_context, assembler->Int32Constant(Context::NATIVE_CONTEXT_INDEX),
- native_context, SKIP_WRITE_BARRIER);
+ assembler->StoreFixedArrayElement(function_context,
+ Context::NATIVE_CONTEXT_INDEX,
+ native_context, SKIP_WRITE_BARRIER);
// Initialize the rest of the slots to undefined.
Node* undefined = assembler->UndefinedConstant();

Powered by Google App Engine
This is Rietveld 408576698