[PATCH] D92274: [lld/mac] Add --reproduce option
Jez Ng via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Sun Nov 29 21:41:45 PST 2020
int3 accepted this revision.
int3 added a comment.
This revision is now accepted and ready to land.
lgtm
================
Comment at: lld/Common/Reproduce.cpp:61
+ v.push_back(' ');
+ v+= quote(arg.getValue(i));
+ }
----------------
thakis wrote:
> (Turns out neither ELF nor COFF ports have MultiArg<> flags. Covered by the `-platform_version` flag in the test.)
================
Comment at: lld/MachO/DriverUtils.cpp:118
+ case OPT_o:
+ os << "-o " << quote(sys::path::filename(arg->getValue())) << "\n";
+ break;
----------------
================
Comment at: lld/MachO/InputFiles.h:40
+// If -reproduce option is given, all input files are written
+// to this tar archive.
----------------
================
Comment at: lld/test/MachO/reproduce.s:33
+# RSP2-NOT: {{^}}repro2{{[/\\]}}
+# RSP2: {{[/\\]}}foo.o
+
----------------
nit: align with previous line
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D92274/new/
https://reviews.llvm.org/D92274
More information about the llvm-commits
mailing list