On Fri, 19 Dec 2025 08:34:05 +1200 Linus Torvalds <[email protected]> wrote:
> On Fri, 19 Dec 2025 at 08:26, David Laight <[email protected]> > wrote: > > > > One possibility is to conditionally add _Pragma() > > No. That compiler warning is pure and utter garbage. I have pointed it > out fopr *years*, and compiler people don't get it. > > So that warning just needs to die. It's shit. It's wrong. True - especially for code like: if (x < 0 || x > limit) return ... where the code is correct even with 'accidental' conversion of a negative signed value to a large unsigned one. clang seems to have a dozen similar warnings, all of which are a PITA for kernel code - like rejecting !(4 << 16). _Pragma() might be usable for -Wshadow, which is generally useful for local variables (but not global functions like log() and j0()). (I usually enable it and fix up the consequences.) Things like the masked userspace access define which carefully creates a readonly variable that shadows a user local would need to disable that one. David
