ARMmbed / nrf51-sdk

Module to contain files provided by the nordic nRF51 SDK
Other
13 stars 17 forks source link

Optimise to omit frame pointer even in a debug build #19

Closed LiyouZhou closed 8 years ago

LiyouZhou commented 8 years ago

In bootloader_util.c there is a piece of asm that uses r7. r7 is usually used by gcc as a frame pointer hence gcc would complain even though a frame pointer is not actually needed. -fomit-frame-pointer prevents this warning.

fixes #11

@pan- @andresag01 please review

andresag01 commented 8 years ago

This seems to fix the problem. I tried building with GCC and -d and it succeeds. Also the application works as expected.

Lets wait for @pan- to comment.

pan- commented 8 years ago

It's ok for me, I merge.