[PATCH] D41180: [dsymutil] Re-enable threading

[Jonas Devlieghere via Phabricator via llvm-commits]

JDevlieghere marked 4 inline comments as done.
JDevlieghere added a comment.

I didn't see you had requested changes between seeing Adrian's accept and actually landing it. I'm happy to address this in a follow-up commit!



================
Comment at: test/tools/dsymutil/X86/alias.test:2
 # RUN: llvm-dsymutil -f -oso-prepend-path=%p/../Inputs/alias \
-# RUN: %p/../Inputs/alias/foobar -o - 2>&1 | llvm-dwarfdump - | FileCheck %s
+# RUN: %p/../Inputs/alias/foobar -o - | llvm-dwarfdump - 2>&1 | FileCheck %s
 # CHECK-NOT: could not find object file symbol for symbol
----------------
davide wrote:
> do you really need to use a relative path here?
> Also, do you need `-o -` even thoughh you're piping to another tool?
How else would you specify the path?

Omitting `-o -` would not pipe the output to stdout but rather create `inputfile.dwarf`. 


Repository:
  rL LLVM

https://reviews.llvm.org/D41180