[llvm] Fix parentheses for clarity in X86ISelLowering.cpp (PR #121954)

via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 9 02:29:31 PST 2025


soumyaGhoshh wrote:

> > okay got it. i think i should give it a try again with cloning entirely
> 
> Although it's a good practice to learn how to build the entire LLVM, and usually it's necessary for committing a patch, you can sometimes verify the problem without entire build. E.g., https://godbolt.org/z/719WE69ex

ahh wow that's a very good idea. it was super helpful thanks, could you tell how you know what specific type or version of compiler to use for checking  or is it essentially the same for all? 
also what you did is that check the line of code generating the warning from the large code file also we need to write the function and it's dependencies right ? 

https://github.com/llvm/llvm-project/pull/121954


More information about the llvm-commits mailing list