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.

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

Jump to bottom

[Silabs] Fix compilation issues with gcc 12.2 #26090

Merged
merged 4 commits into from
Apr 13, 2023
Merged

[Silabs] Fix compilation issues with gcc 12.2 #26090

merged 4 commits into from
Apr 13, 2023

Conversation

jmartinez-silabs
Copy link
Member

@jmartinez-silabs jmartinez-silabs commented Apr 13, 2023 •
edited
Loading

Also deactivate the error flag on the array parameter warning "-Wno-error=array-parameter" until this can be fixed on a silabs GSDK release

EDIT: the flags disabling is commented out for now as they do not exist in the current gcc version used. Uncomment those two when pw update rolls out

@github-actions
Copy link

PR #26090: Size comparison from a235160 to 2220dd6

Increases (1 build for cc32xx)
platform target config section a235160 2220dd6 change % change
cc32xx lock CC3235SF_LAUNCHXL .debug_info 20326060 20326062 2 0.0
Full report (1 build for cc32xx)
platform target config section a235160 2220dd6 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 20326060 20326062 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

@github-actions
Copy link

PR #26090: Size comparison from a235160 to 52c8a85

Increases (1 build for cc32xx)
platform target config section a235160 52c8a85 change % change
cc32xx lock CC3235SF_LAUNCHXL .debug_info 20326060 20326061 1 0.0
Full report (1 build for cc32xx)
platform target config section a235160 52c8a85 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 20326060 20326061 1 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

@github-actions
Copy link

PR #26090: Size comparison from e1f306f to 3988557

Increases (3 builds for bl702, cc13x2_26x2, psoc6)
platform target config section e1f306f 3988557 change % change
bl702 lighting-app bl702+rpc .debug_info 45222211 45222212 1 0.0
.text 1032218 1032220 2 0.0
cc13x2_26x2 pump-controller-app LP_CC2652R7 (read only) 676819 676827 8 0.0
.text 590720 590728 8 0.0
psoc6 all-clusters cy8ckit_062s2_43012 .debug_info 28112348 28112349 1 0.0
Decreases (9 builds for bl602, bl702, cc13x2_26x2, cc32xx, cyw30739, psoc6)
platform target config section e1f306f 3988557 change % change
bl602 lighting-app bl602 .text 1031444 1031442 -2 -0.0
bl602+rpc .text 1062532 1062530 -2 -0.0
bl702 lighting-app bl702 .debug_info 40731651 40731649 -2 -0.0
.text 959238 959234 -4 -0.0
cc13x2_26x2 all-clusters-minimal-app LP_CC2652R7 (read only) 646647 646639 -8 -0.0
.text 569464 569456 -8 -0.0
pump-controller-app LP_CC2652R7 (read/write) 172772 172764 -8 -0.0
cc32xx lock CC3235SF_LAUNCHXL .debug_info 20326062 20326060 -2 -0.0
cyw30739 light cyw930739m2evb_01 (read/write) 536226 536218 -8 -0.0
.app_xip_area 416804 416796 -8 -0.0
psoc6 all-clusters-minimal cy8ckit_062s2_43012 .debug_info 27571738 27571737 -1 -0.0
lock cy8ckit_062s2_43012 .debug_info 23181540 23181539 -1 -0.0
Full report (26 builds for bl602, bl702, cc13x2_26x2, cc32xx, cyw30739, k32w, linux, psoc6, qpg)
platform target config section e1f306f 3988557 change % change
bl602 lighting-app bl602 (read/write) 1350262 1350262 0 0.0
.bss 87870 87870 0 0.0
.data 9648 9648 0 0.0
.text 1031444 1031442 -2 -0.0
bl602+rpc (read/write) 1396266 1396266 0 0.0
.bss 95918 95918 0 0.0
.data 10040 10040 0 0.0
.text 1062532 1062530 -2 -0.0
bl702 lighting-app bl702 0 0 0 0.0
(read only) 3358 3358 0 0.0
(read/write) 1179171 1179171 0 0.0
.bleromro 6342 6342 0 0.0
.bleromrw 124 124 0 0.0
.boot2 328 328 0 0.0
.bss 64617 64617 0 0.0
.bss_psram 24368 24368 0 0.0
.comment 48 48 0 0.0
.data 3952 3952 0 0.0
.debug_abbrev 1549183 1549183 0 0.0
.debug_aranges 134632 134632 0 0.0
.debug_frame 495000 495000 0 0.0
.debug_info 40731651 40731649 -2 -0.0
.debug_line 5308897 5308897 0 0.0
.debug_loc 3444505 3444505 0 0.0
.debug_ranges 376448 376448 0 0.0
.debug_str 3595825 3595825 0 0.0
.hbn 536 536 0 0.0
.hbn_noinit 260 260 0 0.0
.init 342 342 0 0.0
.init_array 140 140 0 0.0
.psram 0 0 0 0.0
.riscv.attributes 47 47 0 0.0
.rodata 105952 105952 0 0.0
.rsvd 2960 2960 0 0.0
.sha_ocram 72 72 0 0.0
.shstrtab 304 304 0 0.0
.stack 2048 2048 0 0.0
.strtab 573386 573386 0 0.0
.symtab 174192 174192 0 0.0
.tcm_data 28 28 0 0.0
.tcmcode 3358 3358 0 0.0
.text 959238 959234 -4 -0.0
bl702+rpc 0 0 0 0.0
(read only) 3358 3358 0 0.0
(read/write) 1267335 1267335 0 0.0
.bleromro 6342 6342 0 0.0
.bleromrw 124 124 0 0.0
.boot2 328 328 0 0.0
.bss 72665 72665 0 0.0
.bss_psram 24768 24768 0 0.0
.comment 48 48 0 0.0
.data 4504 4504 0 0.0
.debug_abbrev 1699693 1699693 0 0.0
.debug_aranges 142840 142840 0 0.0
.debug_frame 522832 522832 0 0.0
.debug_info 45222211 45222212 1 0.0
.debug_line 5716029 5716029 0 0.0
.debug_loc 3645019 3645019 0 0.0
.debug_ranges 401424 401424 0 0.0
.debug_str 4020050 4020050 0 0.0
.hbn 536 536 0 0.0
.hbn_noinit 260 260 0 0.0
.init 342 342 0 0.0
.init_array 156 156 0 0.0
.psram 0 0 0 0.0
.riscv.attributes 47 47 0 0.0
.rodata 119808 119808 0 0.0
.rsvd 2960 2960 0 0.0
.sha_ocram 72 72 0 0.0
.shstrtab 304 304 0 0.0
.stack 2048 2048 0 0.0
.strtab 634387 634387 0 0.0
.symtab 192592 192592 0 0.0
.tcm_data 28 28 0 0.0
.tcmcode 3358 3358 0 0.0
.text 1032218 1032220 2 0.0
cc13x2_26x2 all-clusters-app LP_CC2652R7 (read only) 683367 683367 0 0.0
(read/write) 167896 167896 0 0.0
.bss 80956 80956 0 0.0
.data 3336 3336 0 0.0
.rodata 87071 87071 0 0.0
.text 595980 595980 0 0.0
all-clusters-minimal-app LP_CC2652R7 (read only) 646647 646639 -8 -0.0
(read/write) 157608 157608 0 0.0
.bss 80156 80156 0 0.0
.data 3336 3336 0 0.0
.rodata 76863 76863 0 0.0
.text 569464 569456 -8 -0.0
lock-ftd LP_CC2652R7 (read only) 679199 679199 0 0.0
(read/write) 169512 169512 0 0.0
.bss 78492 78492 0 0.0
.data 3300 3300 0 0.0
.rodata 75751 75751 0 0.0
.text 602968 602968 0 0.0
lock-mtd LP_CC2652R7 (read only) 667051 667051 0 0.0
(read/write) 176836 176836 0 0.0
.bss 73668 73668 0 0.0
.data 3300 3300 0 0.0
.rodata 102635 102635 0 0.0
.text 563936 563936 0 0.0
pump-app LP_CC2652R7 (read only) 691931 691931 0 0.0
(read/write) 157524 157524 0 0.0
.bss 78468 78468 0 0.0
.data 3264 3264 0 0.0
.rodata 89867 89867 0 0.0
.text 601584 601584 0 0.0
pump-controller-app LP_CC2652R7 (read only) 676819 676827 8 0.0
(read/write) 172772 172764 -8 -0.0
.bss 78604 78604 0 0.0
.data 3288 3288 0 0.0
.rodata 85619 85619 0 0.0
.text 590720 590728 8 0.0
shell LP_CC2652R7 (read only) 676206 676206 0 0.0
(read/write) 177128 177128 0 0.0
.bss 83028 83028 0 0.0
.data 3332 3332 0 0.0
.rodata 85358 85358 0 0.0
.text 590536 590536 0 0.0
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
cyw30739 light cyw930739m2evb_01 (read/write) 536226 536218 -8 -0.0
.app_xip_area 416804 416796 -8 -0.0
.bss 61856 61856 0 0.0
.data 724 724 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
lock cyw930739m2evb_01 (read/write) 540602 540602 0 0.0
.app_xip_area 415892 415892 0 0.0
.bss 67136 67136 0 0.0
.data 732 732 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
ota-requestor-no-progress-logging cyw930739m2evb_01 (read/write) 557462 557462 0 0.0
.app_xip_area 438800 438800 0 0.0
.bss 61144 61144 0 0.0
.data 680 680 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
k32w contact k32w0+release (read/write) 676512 676512 0 0.0
.bss 65540 65540 0 0.0
.data 2208 2208 0 0.0
.text 577364 577364 0 0.0
light k32w0+release (read/write) 675652 675652 0 0.0
.bss 65204 65204 0 0.0
.data 2196 2196 0 0.0
.text 576852 576852 0 0.0
lock k32w0+release (read/write) 630384 630384 0 0.0
.bss 63276 63276 0 0.0
.data 2120 2120 0 0.0
.text 549972 549972 0 0.0
linux chip-tool-ipv6only arm64 (read only) 1212170 1212170 0 0.0
(read/write) 742904 742904 0 0.0
.bss 34392 34392 0 0.0
.data 3008 3008 0 0.0
.data.rel.ro 684760 684760 0 0.0
.dynamic 560 560 0 0.0
.got 15528 15528 0 0.0
.init 24 24 0 0.0
.init_array 216 216 0 0.0
.rodata 566804 566804 0 0.0
.text 9791860 9791860 0 0.0
thermostat-no-ble arm64 (read only) 2501564 2501564 0 0.0
(read/write) 145272 145272 0 0.0
.bss 56344 56344 0 0.0
.data 1784 1784 0 0.0
.data.rel.ro 77736 77736 0 0.0
.dynamic 560 560 0 0.0
.got 5360 5360 0 0.0
.init 24 24 0 0.0
.init_array 432 432 0 0.0
.rodata 129496 129496 0 0.0
.text 2109488 2109488 0 0.0
psoc6 all-clusters cy8ckit_062s2_43012 0 0 0 0.0
(read only) 840840 840840 0 0.0
(read/write) 1760772 1760772 0 0.0
.ARM.attributes 46 46 0 0.0
.ARM.exidx 8 8 0 0.0
.bss 189864 189864 0 0.0
.comment 200 200 0 0.0
.copy.table 24 24 0 0.0
.cy_m0p_image 6216 6216 0 0.0
.cy_sharedmem 8 8 0 0.0
.data 2640 2640 0 0.0
.debug_abbrev 1259913 1259913 0 0.0
.debug_aranges 111968 111968 0 0.0
.debug_frame 376964 376964 0 0.0
.debug_info 28112348 28112349 1 0.0
.debug_line 3830148 3830148 0 0.0
.debug_loc 3737219 3737219 0 0.0
.debug_ranges 369952 369952 0 0.0
.debug_str 3553377 3553377 0 0.0
.heap 840840 840840 0 0.0
.noinit 148 148 0 0.0
.ramVectors 736 736 0 0.0
.shstrtab 288 288 0 0.0
.stab 156 156 0 0.0
.stabstr 335 335 0 0.0
.stack_dummy 4096 4096 0 0.0
.strtab 582995 582995 0 0.0
.symtab 426704 426704 0 0.0
.text 1559880 1559880 0 0.0
.zero.table 8 8 0 0.0
all-clusters-minimal cy8ckit_062s2_43012 0 0 0 0.0
(read only) 841648 841648 0 0.0
(read/write) 1700820 1700820 0 0.0
.ARM.attributes 46 46 0 0.0
.ARM.exidx 8 8 0 0.0
.bss 189064 189064 0 0.0
.comment 200 200 0 0.0
.copy.table 24 24 0 0.0
.cy_m0p_image 6216 6216 0 0.0
.cy_sharedmem 8 8 0 0.0
.data 2632 2632 0 0.0
.debug_abbrev 1241777 1241777 0 0.0
.debug_aranges 111120 111120 0 0.0
.debug_frame 379048 379048 0 0.0
.debug_info 27571738 27571737 -1 -0.0
.debug_line 3833551 3833551 0 0.0
.debug_loc 3719289 3719289 0 0.0
.debug_ranges 367800 367800 0 0.0
.debug_str 3539296 3539296 0 0.0
.heap 841648 841648 0 0.0
.noinit 148 148 0 0.0
.ramVectors 736 736 0 0.0
.shstrtab 288 288 0 0.0
.stab 156 156 0 0.0
.stabstr 335 335 0 0.0
.stack_dummy 4096 4096 0 0.0
.strtab 542197 542197 0 0.0
.symtab 411520 411520 0 0.0
.text 1500736 1500736 0 0.0
.zero.table 8 8 0 0.0
light cy8ckit_062s2_43012 0 0 0 0.0
(read only) 849912 849912 0 0.0
(read/write) 1614564 1614564 0 0.0
.ARM.attributes 46 46 0 0.0
.ARM.exidx 8 8 0 0.0
.bss 180968 180968 0 0.0
.comment 200 200 0 0.0
.copy.table 24 24 0 0.0
.cy_m0p_image 6216 6216 0 0.0
.cy_sharedmem 8 8 0 0.0
.data 2464 2464 0 0.0
.debug_abbrev 1077052 1077052 0 0.0
.debug_aranges 103328 103328 0 0.0
.debug_frame 349328 349328 0 0.0
.debug_info 22930065 22930065 0 0.0
.debug_line 3384399 3384399 0 0.0
.debug_loc 3393990 3393990 0 0.0
.debug_ranges 326264 326264 0 0.0
.debug_str 3337962 3337962 0 0.0
.heap 849912 849912 0 0.0
.noinit 148 148 0 0.0
.ramVectors 736 736 0 0.0
.shstrtab 288 288 0 0.0
.stab 156 156 0 0.0
.stabstr 335 335 0 0.0
.stack_dummy 4096 4096 0 0.0
.strtab 477284 477284 0 0.0
.symtab 379152 379152 0 0.0
.text 1422744 1422744 0 0.0
.zero.table 8 8 0 0.0
lock cy8ckit_062s2_43012 0 0 0 0.0
(read only) 822136 822136 0 0.0
(read/write) 1671724 1671724 0 0.0
.ARM.attributes 46 46 0 0.0
.ARM.exidx 8 8 0 0.0
.bss 208728 208728 0 0.0
.comment 200 200 0 0.0
.copy.table 24 24 0 0.0
.cy_m0p_image 6216 6216 0 0.0
.cy_sharedmem 8 8 0 0.0
.data 2480 2480 0 0.0
.debug_abbrev 1079284 1079284 0 0.0
.debug_aranges 103728 103728 0 0.0
.debug_frame 351020 351020 0 0.0
.debug_info 23181540 23181539 -1 -0.0
.debug_line 3386150 3386150 0 0.0
.debug_loc 3417891 3417891 0 0.0
.debug_ranges 328864 328864 0 0.0
.debug_str 3356523 3356523 0 0.0
.heap 822136 822136 0 0.0
.noinit 148 148 0 0.0
.ramVectors 736 736 0 0.0
.shstrtab 288 288 0 0.0
.stab 156 156 0 0.0
.stabstr 335 335 0 0.0
.stack_dummy 4096 4096 0 0.0
.strtab 480681 480681 0 0.0
.symtab 381344 381344 0 0.0
.text 1452128 1452128 0 0.0
.zero.table 8 8 0 0.0
qpg lighting-app qpg6105+debug (read/write) 1154828 1154828 0 0.0
.bss 97404 97404 0 0.0
.data 836 836 0 0.0
.text 601924 601924 0 0.0
lock-app qpg6105+debug (read/write) 1124908 1124908 0 0.0
.bss 92548 92548 0 0.0
.data 840 840 0 0.0
.text 572008 572008 0 0.0

@andy31415 andy31415 merged commit 5568b70 into project-chip:master Apr 13, 2023
@jmartinez-silabs jmartinez-silabs deleted the fix/array-parameter branch April 14, 2023 00:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bzbarsky-apple bzbarsky-apple bzbarsky-apple approved these changes

@andy31415 andy31415 andy31415 approved these changes

@amitnj amitnj Awaiting requested review from amitnj

@anush-apple anush-apple Awaiting requested review from anush-apple

@arkq arkq Awaiting requested review from arkq

@carol-apple carol-apple Awaiting requested review from carol-apple

@chrisdecenzo chrisdecenzo Awaiting requested review from chrisdecenzo

@chshu chshu Awaiting requested review from chshu

@chulspro chulspro Awaiting requested review from chulspro

@cliffamzn cliffamzn Awaiting requested review from cliffamzn

@Damian-Nordic Damian-Nordic Awaiting requested review from Damian-Nordic

@dhrishi dhrishi Awaiting requested review from dhrishi

@electrocucaracha electrocucaracha Awaiting requested review from electrocucaracha

@emargolis emargolis Awaiting requested review from emargolis

@franck-apple franck-apple Awaiting requested review from franck-apple

@gjc13 gjc13 Awaiting requested review from gjc13

@harimau-qirex harimau-qirex Awaiting requested review from harimau-qirex

@harsha-rajendran harsha-rajendran Awaiting requested review from harsha-rajendran

@hawk248 hawk248 Awaiting requested review from hawk248

@jelderton jelderton Awaiting requested review from jelderton

@jepenven-silabs jepenven-silabs Awaiting requested review from jepenven-silabs

@jmeg-sfy jmeg-sfy Awaiting requested review from jmeg-sfy

@joonhaengHeo joonhaengHeo Awaiting requested review from joonhaengHeo

@jtung-apple jtung-apple Awaiting requested review from jtung-apple

@kkasperczyk-no kkasperczyk-no Awaiting requested review from kkasperczyk-no

@ksperling-apple ksperling-apple Awaiting requested review from ksperling-apple

@lazarkov lazarkov Awaiting requested review from lazarkov

@lpbeliveau-silabs lpbeliveau-silabs Awaiting requested review from lpbeliveau-silabs

@LuDuda LuDuda Awaiting requested review from LuDuda

@mlepage-google mlepage-google Awaiting requested review from mlepage-google

@mrjerryjohns mrjerryjohns Awaiting requested review from mrjerryjohns

@msandstedt msandstedt Awaiting requested review from msandstedt

@nivi-apple nivi-apple Awaiting requested review from nivi-apple

@robszewczyk robszewczyk Awaiting requested review from robszewczyk

@saurabhst saurabhst Awaiting requested review from saurabhst

@selissia selissia Awaiting requested review from selissia

@sharadb-amazon sharadb-amazon Awaiting requested review from sharadb-amazon

@tcarmelveilleux tcarmelveilleux Awaiting requested review from tcarmelveilleux

@tecimovic tecimovic Awaiting requested review from tecimovic

@tehampson tehampson Awaiting requested review from tehampson

@vijs vijs Awaiting requested review from vijs

@vivien-apple vivien-apple Awaiting requested review from vivien-apple

@woody-apple woody-apple Awaiting requested review from woody-apple

@xylophone21 xylophone21 Awaiting requested review from xylophone21

@younghak-hwang younghak-hwang Awaiting requested review from younghak-hwang

@yufengwangca yufengwangca Awaiting requested review from yufengwangca

@yunhanw-google yunhanw-google Awaiting requested review from yunhanw-google

Assignees
No one assigned
Labels
examples platform review - approved
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@jmartinez-silabs @andy31415 @bzbarsky-apple @restyled-commits