This warning triggers many times while building world. Downgrade it to
a warning until all occurrences have been fixed. Once the Clang warnings
have been fixed we should be able to turn it on for GCC as well.
See also f4fed768bba45a406f73ed1491d7e52fd1a8711d which did the same for
the kernel builds.
Details
Details
Diff Detail
Diff Detail
- Repository
- rS FreeBSD src repository - subversion
- Lint
Lint Passed - Unit
No Test Coverage - Build Status
Buildable 41477 Build 38366: arc lint + arc unit
Event Timeline
Comment Actions
I did this slightly differently in https://github.com/DimitryAndric/freebsd-src/commit/73a8f96dc7c12ebf76f5a40a7f1208c11f75741f, and I put it in another if block. It would be nice to choose either my version or yours, and integrate it right away, so I don't get needless merge conflicts the next time I sync up the llvm-13-update branch with main.
Comment Actions
Making it an error for WARNS=6 seems reasonable. Happy to just use you version instead.
Comment Actions
Either one is fine with me, and agree we should get it in HEAD immediately, independent of the merge of Clang 13.