[PATCH] D116833: [clang] Introduce support for disabling warnings in system macros

Eli Friedman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jan 10 14:21:31 PST 2022


efriedma added a comment.

I'll just note here that doing this globally is likely to have unexpected results... consider, for example:

  #include <math.h>
  void f() { long x = M_PI; }

Currently, the implicit conversion warning points into math.h.

That said, I don't see any problem with the current implementation.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D116833



More information about the cfe-commits mailing list