[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