[PATCH] D85324: [SystemZ][z/OS] Add z/OS Target and define macros

Hubert Tong via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Aug 21 11:49:34 PDT 2020


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

LGTM from my end; although @MaskRay might want another look.



================
Comment at: clang/lib/Basic/Targets/OSTargets.h:758
+      Builder.defineMacro("__DLL__");
+      // XOPEN_SOURCE=600 is required to build libcxx.
+      Builder.defineMacro("_XOPEN_SOURCE", "600");
----------------
Minor nit: s/XOPEN_SOURCE/_XOPEN_SOURCE/;


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D85324



More information about the cfe-commits mailing list