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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
