[PATCH] D121868: [cc1as] Add support for emitting the build version load command for -darwin-target-variant
Nico Weber via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Apr 22 05:31:17 PDT 2022
thakis added a comment.
Comment at: clang/test/Misc/cc1as-darwin-target-variant-triple.s:2
+// Run cc1as using darwin-target-variant-triple
+// RUN: %clang -cc1as -triple x86_64-apple-macos10.9 -darwin-target-variant-triple x86_64-apple-ios13.1-macabi -filetype obj %s -o - \
+// RUN: | llvm-readobj --file-headers --macho-version-min - \
> thakis wrote:
> > bc-lee wrote:
> > > MaskRay wrote:
> > > > `-filetype obj` on the `%clang -cc1as` line seems weird.
> > > Without this, I cannot create object file and validate it using `llvm-readobj`.
> > If you use `emit-obj` I think this test needs a `REQUIRES: x86-registered-target`.
> > I'm not sure if piping binary data works on Windows – it might have stdout in text mode, which will do some byte replacements.
> A search for `-o -.*\n.*RUN.*llvm-readobj` in the LLVM codebase suggests that there are already several tests using that pattern. Does that really matter?
Well, if it works, it doesn't matter :)
I'll land this and the bots can tell us if it works.
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
More information about the cfe-commits