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

gcc v12 warnings #1013

Open
andrewgait opened this issue Dec 22, 2022 · 0 comments
Open

gcc v12 warnings #1013

andrewgait opened this issue Dec 22, 2022 · 0 comments
Labels
C Relating to the C code in FEC minor Not very important
Milestone

Comments

@andrewgait
Copy link
Contributor

andrewgait commented Dec 22, 2022

Compiling with arm-none-eabi-gcc v12.2.0 gives strange-looking warnings like the following:

arm-none-eabi-gcc -c -mthumb-interwork -march=armv5te -std=gnu99 -ffreestanding -I /Users/user/code/spinnaker/git/spinnaker_tools/include -Wall -Wextra -Wold-style-definition -DPRODUCTION_CODE -DPROFILER_DISABLED -Ofast -DAPPLICATION_NAME_HASH=0xfdeb3b58 -I /Users/user/code/spinnaker/git/SpiNNFrontEndCommon/c_common/models/system_models/modified_src/ -g -o /Users/user/code/spinnaker/git/SpiNNFrontEndCommon/c_common/models/system_models/build/extra_monitor_support.o /Users/user/code/spinnaker/git/SpiNNFrontEndCommon/c_common/models/system_models/modified_src/extra_monitor_support.c
In function ‘sdram_free’,
inlined from ‘data_out_dma_complete_read_missing_seqeuence_nums’ at /Users/user/code/spinnaker/git/SpiNNFrontEndCommon/c_common/models/system_models/modified_src/extra_monitor_support.c:1677:13:
/Users/user/code/spinnaker/git/SpiNNFrontEndCommon/c_common/models/system_models/modified_src/extra_monitor_support.c:584:46: warning: array subscript 0 is outside array bounds of ‘sark_vec_t[0]’ {aka ‘struct sark_vec[]’} [-Warray-bounds]
584 | ALLOC_LOCK | ALLOC_ID | (sark_vec->app_id << 8));
|

It seems from https://gcc.gnu.org/bugzilla/show_bug.cgi?id=105523 as though this is a bug that is yet to be resolved, so I would suggest keeping an eye on the status of that bug to see what happens there.

@andrewgait andrewgait added the minor Not very important label Dec 22, 2022
@dkfellows dkfellows added the C Relating to the C code in FEC label Sep 27, 2023
@dkfellows dkfellows added this to the 7.1.0 milestone Sep 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C Relating to the C code in FEC minor Not very important
Projects
None yet
Development

No branches or pull requests

2 participants