Skip to content

Commit

Permalink
SoC: remove -msave-restore in npk.yml to fix dhrystone benchmark low …
Browse files Browse the repository at this point in the history
  • Loading branch information
fanghuaqi committed Jan 3, 2024
1 parent 5967b2b commit d270370
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 2 deletions.
6 changes: 5 additions & 1 deletion SoC/evalsoc/Common/npk.yml
Original file line number Diff line number Diff line change
Expand Up @@ -378,7 +378,7 @@ debugconfig:
buildconfig:
- type: common
common_flags: # flags need to be combined together across all packages
- flags: -g -fno-common -ffunction-sections -fdata-sections -msave-restore
- flags: -g -fno-common -ffunction-sections -fdata-sections
- flags: -march=${nuclei_arch} -mabi=${nuclei_core.abi}
- flags: -mcmodel=${nuclei_core.cmodel}
- flags: -nostdinc
Expand Down Expand Up @@ -406,6 +406,10 @@ buildconfig:
condition: $( startswith(${stdclib}, "libncrt") && ${semihost} == 1 )
- flags: -lfileops_uart
condition: $( startswith(${stdclib}, "libncrt") && ${semihost} == 0 )
# Workaround for libncrt possible linking error since ide dont support wrap search library
# Require Nuclei Studio Plugin released in 2024
- flags: -l$(subst(${stdclib},lib,))
condition: $( startswith(${stdclib}, "libncrt") )
- flags: -Wl,--gc-sections -Wl,--check-sections
- flags: -Wl,--defsym=__SMP_CPU_CNT=${nuclei_smp}
condition: $( ${nuclei_smp} > 1 )
Expand Down
6 changes: 5 additions & 1 deletion SoC/gd32vf103/Common/npk.yml
Original file line number Diff line number Diff line change
Expand Up @@ -203,7 +203,7 @@ debugconfig:
buildconfig:
- type: common
common_flags: # flags need to be combined together across all packages
- flags: -g -fno-common -ffunction-sections -fdata-sections -msave-restore
- flags: -g -fno-common -ffunction-sections -fdata-sections
- flags: -march=${nuclei_core.arch} -mabi=${nuclei_core.abi}
- flags: -mcmodel=${nuclei_core.cmodel}
- flags: -nostdinc
Expand Down Expand Up @@ -231,6 +231,10 @@ buildconfig:
condition: $( startswith(${stdclib}, "libncrt") && ${semihost} == 1 )
- flags: -lfileops_uart
condition: $( startswith(${stdclib}, "libncrt") && ${semihost} == 0 )
# Workaround for libncrt possible linking error since ide dont support wrap search library
# Require Nuclei Studio Plugin released in 2024
- flags: -l$(subst(${stdclib},lib,))
condition: $( startswith(${stdclib}, "libncrt") )
- flags: -Wl,--gc-sections -Wl,--check-sections
cflags:
asmflags:
Expand Down

0 comments on commit d270370

Please sign in to comment.