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

Resolve warnings when compiling with GCC 12 and GCC 13.1 #257

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open

Resolve warnings when compiling with GCC 12 and GCC 13.1 #257

wants to merge 8 commits into from
+27 −4

Conversation

rptb1
Copy link
Member

@rptb1 rptb1 commented Jul 23, 2023 •
edited

Working on #256 .

@rptb1 rptb1 self-assigned this Jul 23, 2023
@rptb1
Adding GCC version output to check and record which exact version of …
a9407be 
…GCC is in use. The configure script does not display it.
@rptb1 rptb1 linked an issue Jul 23, 2023 that may be closed by this pull request
error: storing the address of local variable 'stackOut' in '*stackOut.0_1' [-Werror=dangling-pointer=] #256
Open
@rptb1 rptb1 added build Problems with builds and build automation os.li Relates to Linux low risk This work is or would be of low risk of introducing defects. labels Jul 23, 2023
@rptb1 rptb1 mentioned this pull request Jul 23, 2023
Open
@rptb1 rptb1 changed the title Resolve warnings when compiling with GCC 12 Resolve warnings when compiling with GCC 12 and GCC 13.1 Jul 25, 2023
@rptb1 rptb1 requested review from UNAA008 and thejayps July 25, 2023 06:53
@rptb1
Copy link
Member Author

rptb1 commented Sep 10, 2023

GCC 13.2 applied as shown in #256 (comment) still produces the error in #256 .

@thejayps thejayps added the essential Will cause failure to meet customer requirements. Assign resources. label Oct 2, 2023
@thejayps
Copy link
Contributor

thejayps commented Oct 2, 2023

All checks pass but check list is out of date, relevant check is not included #258

@rptb1
Copy link
Member Author

rptb1 commented Nov 6, 2023

We need to ensure the original problem is tested by CI. See #258 or possibly we can use some other "latest" environment.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@UNAA008 UNAA008 Awaiting requested review from UNAA008

@thejayps thejayps Awaiting requested review from thejayps

At least 1 approving review is required to merge this pull request.

Assignees

@rptb1 rptb1

Labels
build Problems with builds and build automation essential Will cause failure to meet customer requirements. Assign resources. low risk This work is or would be of low risk of introducing defects. os.li Relates to Linux
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

error: storing the address of local variable 'stackOut' in '*stackOut.0_1' [-Werror=dangling-pointer=]
2 participants
@rptb1 @thejayps