abhina.sreeskantharajan marked 3 inline comments as done.
abhina.sreeskantharajan added inline comments.
================
Comment at: clang/lib/Basic/Targets/OSTargets.h:743
+ Builder.defineMacro("__BFP__");
+ Builder.defineMacro("__BOOL__");
+ Builder.defineMacro("__LONGNAME__");
----------------
hubert.reinterpretcast wrote:
> Sorry for not catching this earlier, but this also needs a FIXME re: strict
> C89.
I've added a comment here.
================
Comment at: clang/test/Preprocessor/init-zos.c:5
+//
+// S390X-ZOS-CXX:#define _EXT 1
+// S390X-ZOS-C99:#define _ISOC99_SOURCE 1
----------------
hubert.reinterpretcast wrote:
> Should this be `GNUXX`?
I agree this is a better name, I've updated the name to your suggestion.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D85324/new/
https://reviews.llvm.org/D85324
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
