[PATCH] D86707: [SystemZ][z/OS] Adding initial toolchain for z/OS

[Abhina Sree via Phabricator via cfe-commits]

abhina.sreeskantharajan marked an inline comment as done.
abhina.sreeskantharajan added inline comments.


================
Comment at: clang/lib/Driver/ToolChains/ZOS.h:25
+
+  bool isPICDefault() const override { return false; }
+  bool isPIEDefault() const override { return false; }
----------------
hubert.reinterpretcast wrote:
> According to the RFC re: LLVM on z/OS, the initial support in LLVM for z/OS is only for XPLink. My understanding is that all XPLink applications are DLL-enabled. Does being DLL-enabled not imply that the code is position-independent?
> 
> I understand that the value of the `__DLL__` predefined macro from the XL C compiler does not reflect the implicit DLL-enablement of XPLink code; however, I also note that the same compiler claims falsely that `Option NODLL is ignored because option XPLINK is specified` when `-qnodll` does actually suppress the effect of an earlier `-qdll` in causing `__DLL__` to be defined.
This is not always true because we do not require code to be PIC on z/OS, even for XPLink applications. Absolute addresses may be present in code sections for easier access (e.g. in calls to linkages, branch tables). We also may link to libraries that contain non-PIC code.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D86707