[PATCH] D134461: [Clang] Warn when trying to deferencing void pointers in C

Nathan Chancellor via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Sep 26 09:35:58 PDT 2022


nathanchance added a comment.

In D134461#3815458 <https://reviews.llvm.org/D134461#3815458>, @aaron.ballman wrote:

> What do folks think of that idea?

I think that all sounds reasonable to me (although I am far from an authority on these matters). As far as I understand it, the kernel cannot use `-pedantic` due to its liberal use of GNU C extensions so moving those specific constructs to a `-pedantic` version of the warning will be the same as just turning them off altogether, which obviously works fine for us. I am happy to take the suggested changes for a spin against the kernel once a patch is available to see if there are any other interesting places where this warning triggers to make sure it does not need further adjustment, since we openly welcome new diagnostics. Right now, it is pretty much impossible to sift through them all because of how often it fires.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D134461/new/

https://reviews.llvm.org/D134461



More information about the cfe-commits mailing list