[PATCH] D126669: [Driver][Modules] Remove dependence on linking support from clang/test/Driver/modules.cpp

[Daniel McIntosh via Phabricator via cfe-commits]

DanielMcIntosh-IBM added a comment.

In D126669#3546172 <https://reviews.llvm.org/D126669#3546172>, @MaskRay wrote:

> This change is fine (if I were adding the tests in the first place, I'd add -c or -S), but I'd like to understand why you have such a toolchain not supporting linking.
> Such special requirements make me feel odd.

Downstream we have a branch that only contains some of the required changes for z/OS support, but which we still run tests on (why this is the case is a long story, but it's something we're working on fixing, it's just going to take a very long time to do so). This test is failing on that branch. Rather than loose the test, I figured I'd improve the test and in the process get it working again on that branch.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D126669