Skip to content

perf(build): enable release O2 optimization #124

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
zjwhitehead wants to merge 1 commit into
base: master
Choose a base branch
from
Open

perf(build): enable release O2 optimization #124

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion platformio.ini
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -51,7 +51,7 @@ build_flags =
;-D BLE_PAIR_ON_BOOT
; Keep the unreferenced firmware version marker through linker --gc-sections
-Wl,-u,fw_version_marker
build_type = debug
build_type = release
debug_speed = 12000
debug_tool = esp-builtin
lib_deps =
Expand Down
7 changes: 3 additions & 4 deletions sdkconfig.OpenPPG-CESP32S3-CAN-SP140
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -208,9 +208,9 @@ CONFIG_ARDUHAL_PARTITION_SCHEME="default"
#
# Compiler options
#
CONFIG_COMPILER_OPTIMIZATION_DEFAULT=y
# CONFIG_COMPILER_OPTIMIZATION_DEFAULT is not set
# CONFIG_COMPILER_OPTIMIZATION_SIZE is not set
# CONFIG_COMPILER_OPTIMIZATION_PERF is not set
CONFIG_COMPILER_OPTIMIZATION_PERF=y
# CONFIG_COMPILER_OPTIMIZATION_NONE is not set
CONFIG_COMPILER_OPTIMIZATION_ASSERTIONS_ENABLE=y
# CONFIG_COMPILER_OPTIMIZATION_ASSERTIONS_SILENT is not set
Expand Down Expand Up @@ -1080,7 +1080,6 @@ CONFIG_FREERTOS_TIMER_QUEUE_LENGTH=10
CONFIG_FREERTOS_QUEUE_REGISTRY_SIZE=0
# CONFIG_FREERTOS_USE_TRACE_FACILITY is not set
# CONFIG_FREERTOS_GENERATE_RUN_TIME_STATS is not set
CONFIG_FREERTOS_TASK_FUNCTION_WRAPPER=y
CONFIG_FREERTOS_CHECK_MUTEX_GIVEN_BY_OWNER=y
# CONFIG_FREERTOS_CHECK_PORT_CRITICAL_COMPLIANCE is not set
# CONFIG_FREERTOS_PLACE_FUNCTIONS_INTO_FLASH is not set
Expand Down Expand Up @@ -1733,7 +1732,7 @@ CONFIG_MONITOR_BAUD_115200B=y
# CONFIG_MONITOR_BAUD_OTHER is not set
CONFIG_MONITOR_BAUD_OTHER_VAL=115200
CONFIG_MONITOR_BAUD=115200
CONFIG_COMPILER_OPTIMIZATION_LEVEL_DEBUG=y
# CONFIG_COMPILER_OPTIMIZATION_LEVEL_DEBUG is not set
# CONFIG_COMPILER_OPTIMIZATION_LEVEL_RELEASE is not set
CONFIG_OPTIMIZATION_ASSERTIONS_ENABLED=y
# CONFIG_OPTIMIZATION_ASSERTIONS_SILENT is not set
Expand Down
5 changes: 4 additions & 1 deletion sdkconfig.defaults
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -28,11 +28,14 @@ CONFIG_ESP_TASK_WDT_PANIC=y
# field crash backtrace can be pulled over USB (espcoredump.py) during testing.
CONFIG_ESP_COREDUMP_ENABLE_TO_FLASH=y
CONFIG_ESP_COREDUMP_DATA_FORMAT_ELF=y
# Optimize production firmware for runtime performance. Assertions remain
# enabled at level 2 in the environment-specific sdkconfig.
CONFIG_COMPILER_OPTIMIZATION_PERF=y
# Watchdog scope: only the throttle control loop + its watchdog task explicitly
# subscribe to the Task WDT (see main.cpp). Do NOT also watch the idle tasks —
# with idle-task checking on, ANY non-safety task (UI/BMS/BLE/a cache-disabled
# NVS commit) that monopolizes a core for >5 s would panic-reboot a *flying*
# device, which the design explicitly forbids. A genuine throttle-loop stall
# still reboots via its own esp_task_wdt_add subscription.
CONFIG_ESP_TASK_WDT_CHECK_IDLE_TASK_CPU0=n
CONFIG_ESP_TASK_WDT_CHECK_IDLE_TASK_CPU1=n
CONFIG_ESP_TASK_WDT_CHECK_IDLE_TASK_CPU1=n
Loading