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

Unified Diff: src/arm/macro-assembler-arm.cc

Issue 12087132: ARM: Add comments to clarify conditionally executed code dealing with d16-d31 (Closed) Base URL: https://v8.googlecode.com/svn/branches/bleeding_edge
Patch Set: Created 7 years, 11 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/deoptimizer-arm.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/arm/macro-assembler-arm.cc
diff --git a/src/arm/macro-assembler-arm.cc b/src/arm/macro-assembler-arm.cc
index 369b406eceb0243ad23c5eb5746a16771e316a3d..2a543c37ccd6e03fc79eb90b6bd41b04986f96e6 100644
--- a/src/arm/macro-assembler-arm.cc
+++ b/src/arm/macro-assembler-arm.cc
@@ -887,6 +887,8 @@ void MacroAssembler::EnterExitFrame(bool save_doubles, int stack_space) {
// Check CPU flags for number of registers, setting the Z condition flag.
CheckFor32DRegs(ip);
+ // Push registers d0-d15, and possibly d16-d31, on the stack.
+ // If d16-d31 are not pushed, decrease the stack pointer instead.
vstm(db_w, sp, d16, d31, ne);
sub(sp, sp, Operand(16 * kDoubleSize), LeaveCC, eq);
vstm(db_w, sp, d0, d15);
@@ -953,6 +955,8 @@ void MacroAssembler::LeaveExitFrame(bool save_doubles,
// Check CPU flags for number of registers, setting the Z condition flag.
CheckFor32DRegs(ip);
+ // Pop registers d0-d15, and possibly d16-d31, from r3.
+ // If d16-d31 are not popped, increase r3 instead.
vldm(ia_w, r3, d0, d15);
vldm(ia_w, r3, d16, d31, ne);
add(r3, r3, Operand(16 * kDoubleSize), LeaveCC, eq);
« no previous file with comments | « src/arm/deoptimizer-arm.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698