[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