[PATCH] D112594: [lld-macho] Implement -S

[Jez Ng via Phabricator via llvm-commits]

int3 accepted this revision.
int3 added a comment.
This revision is now accepted and ready to land.

Thanks!



================
Comment at: lld/MachO/InputFiles.cpp:839-840
 void ObjFile::parseDebugInfo() {
+  if (config->omitDebugInfo)
+    return;
+
----------------
When we implement https://bugs.llvm.org/show_bug.cgi?id=49282, we'll want to parse debug info regardless


================
Comment at: lld/test/MachO/stabs.s:64
+## Check that we don't emit DWARF or stabs when -S is used
+# RUN: cd %t && %lld -lSystem test.o foo.o no-debug.o -S -o %t/test-no-debug
+## grep returns an exit code of 1 if it cannot match the intended pattern. We
----------------
we're not testing `-oso_prefix` here, so there's no need to do the `cd %t`


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D112594