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

Unified Diff: src/arm/full-codegen-arm.cc

Issue 1758003: Changed inlined property load detection on ARM... (Closed) Base URL: http://v8.googlecode.com/svn/branches/bleeding_edge/
Patch Set: '' Created 10 years, 8 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/arm/codegen-arm.cc ('k') | src/arm/ic-arm.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/arm/full-codegen-arm.cc
===================================================================
--- src/arm/full-codegen-arm.cc (revision 4479)
+++ src/arm/full-codegen-arm.cc (working copy)
@@ -700,12 +700,7 @@
__ push(ip);
__ mov(r2, Operand(var->name()));
Handle<Code> ic(Builtins::builtin(Builtins::LoadIC_Initialize));
- { Assembler::BlockConstPoolScope block_const_pool(masm_);
- __ Call(ic, RelocInfo::CODE_TARGET_CONTEXT);
- // A B instruction following the call signals that the load was inlined.
- // Ensure that there is not a B instruction here.
- __ nop();
- }
+ __ Call(ic, RelocInfo::CODE_TARGET_CONTEXT);
DropAndApply(1, context, r0);
} else if (slot != NULL && slot->type() == Slot::LOOKUP) {
@@ -1003,12 +998,7 @@
Literal* key = prop->key()->AsLiteral();
__ mov(r2, Operand(key->handle()));
Handle<Code> ic(Builtins::builtin(Builtins::LoadIC_Initialize));
- { Assembler::BlockConstPoolScope block_const_pool(masm_);
- __ Call(ic, RelocInfo::CODE_TARGET);
- // A B instruction following the call signals that the load was inlined.
- // Ensure that there is not a B instruction here.
- __ nop();
- }
+ __ Call(ic, RelocInfo::CODE_TARGET);
}
@@ -1445,12 +1435,7 @@
Handle<Code> ic(Builtins::builtin(Builtins::LoadIC_Initialize));
// Use a regular load, not a contextual load, to avoid a reference
// error.
- { Assembler::BlockConstPoolScope block_const_pool(masm_);
- __ Call(ic, RelocInfo::CODE_TARGET);
- // A B instruction following the call signals that the load was
- // inlined. Ensure that there is not a B instruction here.
- __ nop();
- }
+ __ Call(ic, RelocInfo::CODE_TARGET);
__ str(r0, MemOperand(sp));
} else if (proxy != NULL &&
proxy->var()->slot() != NULL &&
« no previous file with comments | « src/arm/codegen-arm.cc ('k') | src/arm/ic-arm.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698