-
Notifications
You must be signed in to change notification settings - Fork 17k
Should #elifdef and #elifndef have a portability warning? #55306
Copy link
Copy link
Closed
Labels
clang:diagnosticsNew/improved warning or error message in Clang, but not in clang-tidy or static analyzerNew/improved warning or error message in Clang, but not in clang-tidy or static analyzerclang:frontendLanguage frontend issues, e.g. anything involving "Sema"Language frontend issues, e.g. anything involving "Sema"good first issuehttps://github.com/llvm/llvm-project/contributehttps://github.com/llvm/llvm-project/contribute
Description
Metadata
Metadata
Assignees
Labels
clang:diagnosticsNew/improved warning or error message in Clang, but not in clang-tidy or static analyzerNew/improved warning or error message in Clang, but not in clang-tidy or static analyzerclang:frontendLanguage frontend issues, e.g. anything involving "Sema"Language frontend issues, e.g. anything involving "Sema"good first issuehttps://github.com/llvm/llvm-project/contributehttps://github.com/llvm/llvm-project/contribute
Type
Fields
Give feedbackNo fields configured for issues without a type.
These are new features of C2x, but they're supported as a conforming extension in older versions of Clang. However, there's no pedantic warning given about them being a Clang extension in older language modes, so users may be surprised when porting to another compiler. See https://godbolt.org/z/MdK476q7b as an example. Should we be issuing a pedantic warning here?