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

Issue 1530043: Ensure stack alignment in ARM port... (Closed)

Created:
10 years, 8 months ago by Søren Thygesen Gjesse
Modified:
9 years, 7 months ago
Reviewers:
Erik Corry
CC:
v8-dev
Visibility:
Public.

Description

Ensure stack alignment in ARM port In the CEntryStub keep track of the actual skew from the correct alignment to ensure proper alignment before calling the runtime. Add checks to test the expected skew when running on hardware. Use the PrepareCallCFunction/CallCFunction in a few other places. Add check to the ARM simulator for correct alignment. Add option --sim-stack-alignment to set the stack alignment in the simulator. Currntly only values 4 and 8 makes sense. 8 is the default to make snapshots generated with the simulator work on hardware requiring 8 byte stack alignment. Committed: http://code.google.com/p/v8/source/detail?r=4425

Patch Set 1 #

Patch Set 2 : '' #

Patch Set 3 : '' #

Patch Set 4 : '' #

Total comments: 2

Patch Set 5 : '' #

Patch Set 6 : '' #

Unified diffs Side-by-side diffs Delta from patch set Stats (+123 lines, -39 lines) Patch
M src/arm/codegen-arm.cc View 1 2 3 4 5 8 chunks +51 lines, -9 lines 0 comments Download
M src/arm/macro-assembler-arm.h View 1 2 3 4 1 chunk +2 lines, -2 lines 0 comments Download
M src/arm/macro-assembler-arm.cc View 1 2 3 4 5 4 chunks +42 lines, -20 lines 0 comments Download
M src/arm/simulator-arm.cc View 1 2 3 4 3 chunks +19 lines, -4 lines 0 comments Download
M src/codegen.h View 1 2 3 4 1 chunk +2 lines, -1 line 0 comments Download
M src/flag-definitions.h View 1 2 3 4 1 chunk +3 lines, -1 line 0 comments Download
M src/ia32/codegen-ia32.cc View 1 2 3 4 1 chunk +2 lines, -1 line 0 comments Download
M src/x64/codegen-x64.cc View 1 2 3 4 1 chunk +2 lines, -1 line 0 comments Download

Messages

Total messages: 3 (0 generated)
Søren Thygesen Gjesse
I am quite sure that the push of 0 in the CEntryStub::GenerateCore might as well ...
10 years, 8 months ago (2010-04-15 08:08:26 UTC) #1
Erik Corry
Don't we need something similar on Intel? Mac has stack alignment requirements in the ABI ...
10 years, 8 months ago (2010-04-15 08:42:14 UTC) #2
Søren Thygesen Gjesse
10 years, 8 months ago (2010-04-15 09:34:22 UTC) #3
http://codereview.chromium.org/1530043/diff/26001/14007
File src/arm/codegen-arm.cc (right):

http://codereview.chromium.org/1530043/diff/26001/14007#newcode7157
src/arm/codegen-arm.cc:7157: // restore sp back to before aligning the stack.
On 2010/04/15 08:42:14, Erik Corry wrote:
> restore -> Restore
> to before -> to its value from before

Done.

Powered by Google App Engine
This is Rietveld 408576698