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

Enable -Wconversion tree-wide. #25362

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from
Draft

Enable -Wconversion tree-wide. #25362

wants to merge 2 commits into from

Conversation

bzbarsky-apple
Copy link
Contributor

No description provided.

@github-actions github-actions bot added the gn label Feb 28, 2023
@github-actions
Copy link

github-actions bot commented Mar 6, 2023

PR #25362: Size comparison from 1d8d73f to b89f25f

Decreases (1 build for cc32xx)
platform target config section 1d8d73f b89f25f change % change
cc32xx lock CC3235SF_LAUNCHXL .debug_info 20262928 20262927 -1 -0.0
Full report (1 build for cc32xx)
platform target config section 1d8d73f b89f25f change % change
cc32xx lock CC3235SF_LAUNCHXL 0 0 0 0.0
(read only) 643465 643465 0 0.0
(read/write) 203688 203688 0 0.0
.ARM.attributes 44 44 0 0.0
.ARM.exidx 8 8 0 0.0
.bss 197088 197088 0 0.0
.comment 194 194 0 0.0
.data 1480 1480 0 0.0
.debug_abbrev 930213 930213 0 0.0
.debug_aranges 87336 87336 0 0.0
.debug_frame 300024 300024 0 0.0
.debug_info 20262928 20262927 -1 -0.0
.debug_line 2657822 2657822 0 0.0
.debug_loc 2800026 2800026 0 0.0
.debug_ranges 282240 282240 0 0.0
.debug_str 3023883 3023883 0 0.0
.ramVecs 780 780 0 0.0
.resetVecs 64 64 0 0.0
.rodata 105929 105929 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 378514 378514 0 0.0
.symtab 256560 256560 0 0.0
.text 535412 535412 0 0.0
@github-actions
Copy link

github-actions bot commented Mar 6, 2023

PR #25362: Size comparison from 80a2c6e to 21dfe86

Increases (1 build for cc32xx)
platform target config section 80a2c6e 21dfe86 change % change
cc32xx lock CC3235SF_LAUNCHXL .debug_info 20262928 20262929 1 0.0
Full report (1 build for cc32xx)
platform target config section 80a2c6e 21dfe86 change % change
cc32xx lock CC3235SF_LAUNCHXL 0 0 0 0.0
(read only) 643465 643465 0 0.0
(read/write) 203688 203688 0 0.0
.ARM.attributes 44 44 0 0.0
.ARM.exidx 8 8 0 0.0
.bss 197088 197088 0 0.0
.comment 194 194 0 0.0
.data 1480 1480 0 0.0
.debug_abbrev 930213 930213 0 0.0
.debug_aranges 87336 87336 0 0.0
.debug_frame 300024 300024 0 0.0
.debug_info 20262928 20262929 1 0.0
.debug_line 2657822 2657822 0 0.0
.debug_loc 2800026 2800026 0 0.0
.debug_ranges 282240 282240 0 0.0
.debug_str 3023883 3023883 0 0.0
.ramVecs 780 780 0 0.0
.resetVecs 64 64 0 0.0
.rodata 105929 105929 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 378514 378514 0 0.0
.symtab 256560 256560 0 0.0
.text 535412 535412 0 0.0
@github-actions github-actions bot removed the tizen For Tizen platform label Mar 7, 2023
@github-actions
Copy link

github-actions bot commented Mar 7, 2023

PR #25362: Size comparison from 69ec3ea to 13c9c7c

Decreases (1 build for cc32xx)
platform target config section 69ec3ea 13c9c7c change % change
cc32xx lock CC3235SF_LAUNCHXL .debug_info 20267068 20267066 -2 -0.0
Full report (3 builds for cc32xx, qpg)
platform target config section 69ec3ea 13c9c7c change % change
cc32xx lock CC3235SF_LAUNCHXL 0 0 0 0.0
(read only) 644425 644425 0 0.0
(read/write) 203688 203688 0 0.0
.ARM.attributes 44 44 0 0.0
.ARM.exidx 8 8 0 0.0
.bss 197088 197088 0 0.0
.comment 194 194 0 0.0
.data 1480 1480 0 0.0
.debug_abbrev 930235 930235 0 0.0
.debug_aranges 87336 87336 0 0.0
.debug_frame 300028 300028 0 0.0
.debug_info 20267068 20267066 -2 -0.0
.debug_line 2659698 2659698 0 0.0
.debug_loc 2802749 2802749 0 0.0
.debug_ranges 282952 282952 0 0.0
.debug_str 3023892 3023892 0 0.0
.ramVecs 780 780 0 0.0
.resetVecs 64 64 0 0.0
.rodata 105929 105929 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 378514 378514 0 0.0
.symtab 256624 256624 0 0.0
.text 536372 536372 0 0.0
qpg lighting-app qpg6105+debug (read/write) 1152020 1152020 0 0.0
.bss 99812 99812 0 0.0
.data 852 852 0 0.0
.text 599116 599116 0 0.0
lock-app qpg6105+debug (read/write) 1119084 1119084 0 0.0
.bss 96292 96292 0 0.0
.data 864 864 0 0.0
.text 566184 566184 0 0.0
@github-actions github-actions bot added the config label Mar 9, 2023
@github-actions
Copy link

github-actions bot commented Mar 9, 2023

PR #25362: Size comparison from 82da16f to 0c0a1ca

Full report (1 build for cc32xx)
platform target config section 82da16f 0c0a1ca change % change
cc32xx lock CC3235SF_LAUNCHXL 0 0 0 0.0
(read only) 644425 644425 0 0.0
(read/write) 203688 203688 0 0.0
.ARM.attributes 44 44 0 0.0
.ARM.exidx 8 8 0 0.0
.bss 197088 197088 0 0.0
.comment 194 194 0 0.0
.data 1480 1480 0 0.0
.debug_abbrev 930235 930235 0 0.0
.debug_aranges 87344 87344 0 0.0
.debug_frame 300044 300044 0 0.0
.debug_info 20267469 20267469 0 0.0
.debug_line 2659770 2659770 0 0.0
.debug_loc 2802853 2802853 0 0.0
.debug_ranges 282960 282960 0 0.0
.debug_str 3024079 3024079 0 0.0
.ramVecs 780 780 0 0.0
.resetVecs 64 64 0 0.0
.rodata 105929 105929 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 378571 378571 0 0.0
.symtab 256624 256624 0 0.0
.text 536372 536372 0 0.0
@stale
Copy link

stale bot commented May 8, 2023

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.
@stale stale bot added the stale Stale issue or PR label May 8, 2023
@stale
Copy link

stale bot commented Jun 9, 2023

This stale pull request has been automatically closed. Thank you for your contributions.
@stale stale bot closed this Jun 9, 2023
@stale stale bot removed the stale Stale issue or PR label Jun 10, 2023
@stale
Copy link

stale bot commented Aug 10, 2023

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.
@stale stale bot added the stale Stale issue or PR label Aug 10, 2023
@bzbarsky-apple bzbarsky-apple removed the stale Stale issue or PR label Aug 10, 2023
@stale
Copy link

stale bot commented Oct 15, 2023

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.
@stale stale bot added the stale Stale issue or PR label Oct 15, 2023
@bzbarsky-apple bzbarsky-apple removed the stale Stale issue or PR label Oct 16, 2023
@woody-apple woody-apple added this to the No Target Milestone milestone Nov 16, 2023
@stale Stale
Copy link

stale bot commented Mar 13, 2024

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.
@stale stale bot added the stale Stale issue or PR label Mar 13, 2024
@bzbarsky-apple bzbarsky-apple removed the stale Stale issue or PR label Mar 13, 2024
@bzbarsky-apple
Copy link
Contributor Author

This is currently blocked on pigweed being broken, but we do in fact want to do this once they get their headers in order.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
config controller examples gn linux platform review - pending setup payload
2 participants
@bzbarsky-apple @woody-apple