Skip to content
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.

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

Update qpg_sdk for GCC 12.2 compiler warnings #26094

Merged

Conversation

tima-q
Copy link
Contributor

@tima-q tima-q commented Apr 13, 2023

Issue:
Pigweed updated his compiler to GCC 12.2 in google/pigweed@6717dc9

Dependabot could not upgrade due to the warnings in our SDK code.
see - #26031

Fix:
SDK code now updated to fix seen warnings

Testing:
Local build with 12.2.mpacbti-rel1.1 version done to check warning-free run.

@github-actions
Copy link

PR #26094: Size comparison from e1f306f to 2167964

Increases above 0.2%:

platform target config section e1f306f 2167964 change % change
qpg lighting-app qpg6105+debug (read/write) 1154828 1166536 11708 1.0
.bss 97404 99308 1904 2.0
.data 836 856 20 2.4
.text 601924 613636 11712 1.9
lock-app qpg6105+debug (read/write) 1124908 1136608 11700 1.0
.bss 92548 94452 1904 2.1
.data 840 860 20 2.4
.text 572008 583704 11696 2.0
Increases (2 builds for qpg)
platform target config section e1f306f 2167964 change % change
qpg lighting-app qpg6105+debug (read/write) 1154828 1166536 11708 1.0
.bss 97404 99308 1904 2.0
.data 836 856 20 2.4
.text 601924 613636 11712 1.9
lock-app qpg6105+debug (read/write) 1124908 1136608 11700 1.0
.bss 92548 94452 1904 2.1
.data 840 860 20 2.4
.text 572008 583704 11696 2.0
Decreases (1 build for cc32xx)
platform target config section e1f306f 2167964 change % change
cc32xx lock CC3235SF_LAUNCHXL .debug_info 20326062 20326060 -2 -0.0
Full report (4 builds for cc32xx, mbed, qpg)
platform target config section e1f306f 2167964 change % change
cc32xx lock CC3235SF_LAUNCHXL 0 0 0 0.0
(read only) 643081 643081 0 0.0
(read/write) 203848 203848 0 0.0
.ARM.attributes 44 44 0 0.0
.ARM.exidx 8 8 0 0.0
.bss 197248 197248 0 0.0
.comment 194 194 0 0.0
.data 1480 1480 0 0.0
.debug_abbrev 933224 933224 0 0.0
.debug_aranges 87760 87760 0 0.0
.debug_frame 302028 302028 0 0.0
.debug_info 20326062 20326060 -2 -0.0
.debug_line 2687403 2687403 0 0.0
.debug_loc 2838361 2838361 0 0.0
.debug_ranges 288040 288040 0 0.0
.debug_str 3042379 3042379 0 0.0
.ramVecs 780 780 0 0.0
.resetVecs 64 64 0 0.0
.rodata 104393 104393 0 0.0
.shstrtab 232 232 0 0.0
.stab 204 204 0 0.0
.stabstr 441 441 0 0.0
.stack 2048 2048 0 0.0
.strtab 377626 377626 0 0.0
.symtab 256832 256832 0 0.0
.text 536568 536568 0 0.0
mbed lock-app CY8CPROTO_062_4343W+release (read only) 6224 6224 0 0.0
(read/write) 2466272 2466272 0 0.0
.bss 215960 215960 0 0.0
.data 5888 5888 0 0.0
.text 1428916 1428916 0 0.0
qpg lighting-app qpg6105+debug (read/write) 1154828 1166536 11708 1.0
.bss 97404 99308 1904 2.0
.data 836 856 20 2.4
.text 601924 613636 11712 1.9
lock-app qpg6105+debug (read/write) 1124908 1136608 11700 1.0
.bss 92548 94452 1904 2.1
.data 840 860 20 2.4
.text 572008 583704 11696 2.0

@andy31415 andy31415 merged commit f31e396 into project-chip:master Apr 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants