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

Add language limit to the option '-Wno-register' in the Telink config #28043

Merged
merged 3 commits into from
Jul 20, 2023
Merged

Add language limit to the option '-Wno-register' in the Telink config #28043

merged 3 commits into from
Jul 20, 2023

Conversation

Semi-Bin-Blain
Copy link
Contributor

the option '-Wno-register' is valid for C++/ObjC++ but not for C

the C object will get a warning from it, like the log shown

[596/669] Building C object CMakeFiles/app.dir/src/main.c.obj
cc1: warning: command-line option '-Wno-register' is valid for C++/ObjC++ but not for C
[597/669] Building C object CMakeFiles/app.dir/src/device_config.c.obj
cc1: warning: command-line option '-Wno-register' is valid for C++/ObjC++ but not for C

…config

the option '-Wno-register' is valid for C++/ObjC++ but not for C
@CLAassistant
Copy link

CLAassistant commented Jul 19, 2023

CLA assistant check
All committers have signed the CLA.

@mergify mergify bot merged commit ee014cb into project-chip:master Jul 20, 2023
erwinpan1 pushed a commit to erwinpan1/connectedhomeip that referenced this pull request Jul 21, 2023
…config (project-chip#28043)

the option '-Wno-register' is valid for C++/ObjC++ but not for C

Co-authored-by: Andrei Litvin <[email protected]>
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.

4 participants