djsundog/circuitpython
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

qemu-arm: Enable gcc LTO option for nlrthumb.c

Browse Source 
LTO can't "see" inside naked functions, but we can mark `nlr_push_tail` as used.
This commit is contained in:
Daniel Tralamazza 2016-07-08 15:46:01 +02:00 committed by Damien George
parent 7711d87563
commit 9626662819
2 changed files with 1 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
py/nlrthumb.c
View File

@ -71,7 +71,7 @@ __attribute__((naked)) unsigned int nlr_push(nlr_buf_t *nlr) {
return 0; // needed to silence compiler warning return 0; // needed to silence compiler warning
} }
unsigned int nlr_push_tail(nlr_buf_t *nlr) { __attribute__((used)) unsigned int nlr_push_tail(nlr_buf_t *nlr) {
nlr_buf_t **top = &MP_STATE_THREAD(nlr_top); nlr_buf_t **top = &MP_STATE_THREAD(nlr_top);
nlr->prev = *top; nlr->prev = *top;
*top = nlr; *top = nlr;

3
qemu-arm/Makefile
View File

@ -53,9 +53,6 @@ OBJ_TEST += $(addprefix $(BUILD)/, $(SRC_TEST_C:.c=.o))
OBJ_TEST += $(addprefix $(BUILD)/, $(SRC_S:.s=.o)) OBJ_TEST += $(addprefix $(BUILD)/, $(SRC_S:.s=.o))
OBJ_TEST += $(BUILD)/tinytest.o OBJ_TEST += $(BUILD)/tinytest.o
# we can't use LTO for nlrthumb.c because it contains inline assembler
$(PY_BUILD)/nlrthumb.o: CFLAGS += -fno-lto
all: run all: run
run: $(BUILD)/firmware.elf run: $(BUILD)/firmware.elf