[PATCH] D106394: [clang][pp] adds '#pragma include_instead'

[Christopher Di Bella via Phabricator via cfe-commits]

cjdb added inline comments.


================
Comment at: clang/include/clang/Basic/DiagnosticLexKinds.td:304
+
+def pp_pragma_include_instead_not_sysheader : Error<
+  "'#pragma clang include_instead' cannot be used outside of system headers">,
----------------
aaron.ballman wrote:
> Can you prefix the diagnostic identifiers with `err_`? We're not super consistent in this file, but it's helpful when reading the use of the diagnostic to know whether it's an error or not.
I've replaced `pp` with `err` since `pragma` implies "preprocessor" and don't really think it adds much more value. `err` on the other hand is useful?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D106394