[PATCH] D85324: [SystemZ][z/OS] Add z/OS Target and define macros
Abhina Sree via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Aug 27 05:08:29 PDT 2020
abhina.sreeskantharajan added inline comments.
================
Comment at: clang/test/Preprocessor/init-zos.c:4
+
+// S390X-ZOS-GNUXX:#define _EXT 1
+// S390X-ZOS:#define _LONG_LONG 1
----------------
MaskRay wrote:
> --match-full-lines is different from --match-full-lines --strict-whitespace. You can freely add leading and trailing spaces to align `#` for readability.
>
> Might not be necessary changing now.
Thanks MaskRay, I will try to update this testcase in upcoming patches that touch this file.
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