[libc-commits] [libc] [libc] [Task] Enable most libc disabled warnings (PR #122835)
Vinay Deshmukh via libc-commits
libc-commits at lists.llvm.org
Wed Jan 15 04:34:30 PST 2025
vinay-deshmukh wrote:
> Thanks for the patch! Have you tested it with both GCC and clang?
>
> It might be worthwhile to break this up into distinct PRs:
>
> 1. enable most warnings that dont require any fixes today.
> 2. for each new warning enabled that requires source fixes, submit a distinct PR
>
> It looks like most of the fixes are -Wunused and -Wconversion?
I have only used AppleClang for this. I was hoping the CI has GCC and LLVM's Clang as well.
To confirm, the first PR would comprise of the discussed CMake fixes && enable the warnings that don't require fixes
I'll raise those in the coming days!
https://github.com/llvm/llvm-project/pull/122835
More information about the libc-commits
mailing list