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

Jonas Devlieghere via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 13 05:58:47 PST 2017


JDevlieghere created this revision.
JDevlieghere added reviewers: aprantl, vsk.
Herald added a subscriber: javed.absar.

Threading was disabled in r317263 because it broke a test in combination
with `-DLLVM_ENABLE_THREADS=OFF`. This was because a ThreadPool warning
was piped to llvm-dwarfdump which was expecting to read an object from
stdin.

This patch re-enables threading and fixes the offending test.
Unfortunately this required more than just moving the ThreadPool out of
the for loop because of the TempFile refactoring that took place in the
meantime.


Repository:
  rL LLVM

https://reviews.llvm.org/D41180

Files:
  test/tools/dsymutil/ARM/fat-threading.test
  test/tools/dsymutil/X86/alias.test
  tools/dsymutil/dsymutil.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D41180.126751.patch
Type: text/x-patch
Size: 5624 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20171213/91c7a023/attachment.bin>


More information about the llvm-commits mailing list