aaron.ballman added inline comments.

================
Comment at: clang/include/clang-c/ExternC.h:18-19
+#define LLVM_CLANG_C_STRING_PROTOTYPES_BEGIN                                   
\
+  _Pragma("clang diagnostic push")                                             
\
+      _Pragma("clang diagnostic error \"-Wstrict-prototypes\"")
+#define LLVM_CLANG_C_STRING_PROTOTYPES_END _Pragma("clang diagnostic pop")
----------------
I think this needs some extra compiler guards, unfortunately. I don't think 
MSVC supports `_Pragma` (IIRC, they rather oddly decided to call it `__pragma` 
instead). Do we want to restrict this pragma to only clang compilers? Do we 
need to do something to support GCC as well?


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

https://reviews.llvm.org/D70285



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to