Skip to content

Commit

Permalink
fix arm stack alignment
Browse files Browse the repository at this point in the history
  • Loading branch information
crabtw committed May 22, 2013
1 parent 15e4438 commit 499b022
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 3 deletions.
4 changes: 3 additions & 1 deletion src/libcore/rt/context.rs
Expand Up @@ -165,7 +165,9 @@ fn new_regs() -> ~Registers { ~([0, .. 32]) }

#[cfg(target_arch = "arm")]
fn initialize_call_frame(regs: &mut Registers, fptr: *c_void, arg: *c_void, sp: *mut uint) {
let sp = mut_offset(sp, -1);
let sp = align_down(sp);
// sp of arm eabi is 8-byte aligned
let sp = mut_offset(sp, -2);

// The final return address. 0 indicates the bottom of the stack
unsafe { *sp = 0; }
Expand Down
6 changes: 4 additions & 2 deletions src/rt/arch/arm/context.cpp
Expand Up @@ -26,9 +26,11 @@ void context::call(void *f, void *arg, void *stack)

// set up the stack
uint32_t *sp = ( uint32_t *)stack;
//sp = align_down(sp);
sp = align_down(sp);
// The final return address. 0 indicates the bottom of the stack
*--sp = 0;
// sp of arm eabi is 8-byte aligned
sp -= 2;
*sp = 0;

regs.data[0] = ( uint32_t )arg; // r0
regs.data[13] = ( uint32_t )sp; //#52 sp, r13
Expand Down

5 comments on commit 499b022

@bors
Copy link
Contributor

@bors bors commented on 499b022 May 22, 2013

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

saw approval from brson
at crabtw@499b022

@bors
Copy link
Contributor

@bors bors commented on 499b022 May 22, 2013

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

merging crabtw/rust/arm-rt = 499b022 into auto

@bors
Copy link
Contributor

@bors bors commented on 499b022 May 22, 2013

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

crabtw/rust/arm-rt = 499b022 merged ok, testing candidate = 8a4bffc

@bors
Copy link
Contributor

@bors bors commented on 499b022 May 22, 2013

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@bors
Copy link
Contributor

@bors bors commented on 499b022 May 22, 2013

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fast-forwarding incoming to auto = 8a4bffc

Please sign in to comment.