[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