[PATCH] D131351: [C] Default implicit function pointer conversions diagnostic to be an error

Corentin Jabot via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sun Aug 7 07:04:16 PDT 2022


cor3ntin accepted this revision.
cor3ntin added a comment.
This revision is now accepted and ready to land.

LGTM.
I agree that this address potential security issues. I'm curious how many people will be affected, but hopefully by doing that early in the clang 16 cycle we might find out!
Thanks for working on this Aarron.



================
Comment at: clang/include/clang/Basic/DiagnosticSemaKinds.td:8178
+def ext_typecheck_convert_incompatible_function_pointer : ExtWarn<
+  err_typecheck_convert_incompatible_function_pointer.Text>,
+  InGroup<IncompatibleFunctionPointerTypes>, DefaultError;
----------------
Fancy!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D131351



More information about the cfe-commits mailing list