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

Code analysis reports clang-diagnostic-error warnings when cfenv is used with g++ 11 #9366

Closed
sean-mcmanus opened this issue May 25, 2022 · 1 comment
Closed

Code analysis reports clang-diagnostic-error warnings when cfenv is used with g++ 11 #9366

sean-mcmanus opened this issue May 25, 2022 · 1 comment
Assignees
@sean-mcmanus
Labels
bug Feature: Code Analysis Related to integration with clang-tidy, cppcheck, cl.exe /analyze, etc. fixed Check the Milestone for the release in which the fix is or will be available. Language Service
Milestone
1.11.1

Comments

@sean-mcmanus
Copy link
Collaborator

The issue doesn't repro with g++ 12. I'm not sure yet if it's our bug or an issue with clang-tidy or g++. I also used C++20 on Ubuntu 22.

i.e. a bunch of warnings like "no member named 'fenv_t' in the global namespace".
@sean-mcmanus sean-mcmanus added bug Language Service Feature: Code Analysis Related to integration with clang-tidy, cppcheck, cl.exe /analyze, etc. labels May 25, 2022
@sean-mcmanus sean-mcmanus added this to the 1.10 milestone May 25, 2022
@sean-mcmanus sean-mcmanus self-assigned this May 25, 2022
@sean-mcmanus sean-mcmanus modified the milestones: 1.10, 1.11 Jun 7, 2022
@sean-mcmanus
Copy link
Collaborator Author

The issue appears to be fixed if we change it to use -isystem instead of -I.

@sean-mcmanus sean-mcmanus modified the milestones: 1.11, 1.11.0 Jun 8, 2022
@sean-mcmanus sean-mcmanus modified the milestones: 1.11.0, 1.11.1 Jun 16, 2022
@sean-mcmanus sean-mcmanus added the fixed Check the Milestone for the release in which the fix is or will be available. label Jun 22, 2022
@github-actions github-actions bot locked and limited conversation to collaborators Aug 23, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@sean-mcmanus sean-mcmanus

Labels
bug Feature: Code Analysis Related to integration with clang-tidy, cppcheck, cl.exe /analyze, etc. fixed Check the Milestone for the release in which the fix is or will be available. Language Service
Projects
None yet
Milestone
1.11.1
Development

No branches or pull requests
1 participant
@sean-mcmanus