[lld] r317632 - Use std::thread directly.

[Rui Ueyama via llvm-commits]

I found that you already did that. Thanks!


On Tue, Nov 7, 2017 at 3:16 PM, Rui Ueyama <ruiu at google.com> wrote:

> Can you remove runBackground because I think this is the last user of the
> function?
>
> On Tue, Nov 7, 2017 at 3:09 PM, Rafael Espindola via llvm-commits <
> llvm-commits at lists.llvm.org> wrote:
>
>> Author: rafael
>> Date: Tue Nov  7 15:09:54 2017
>> New Revision: 317632
>>
>> URL: http://llvm.org/viewvc/llvm-project?rev=317632&view=rev
>> Log:
>> Use std::thread directly.
>>
>> Now that this code is posix only we don't need to use runBackground.
>>
>> Modified:
>>     lld/trunk/ELF/Filesystem.cpp
>>
>> Modified: lld/trunk/ELF/Filesystem.cpp
>> URL: http://llvm.org/viewvc/llvm-project/lld/trunk/ELF/Filesystem
>> .cpp?rev=317632&r1=317631&r2=317632&view=diff
>> ============================================================
>> ==================
>> --- lld/trunk/ELF/Filesystem.cpp (original)
>> +++ lld/trunk/ELF/Filesystem.cpp Tue Nov  7 15:09:54 2017
>> @@ -20,6 +20,7 @@
>>  #if defined(HAVE_UNISTD_H)
>>  #include <unistd.h>
>>  #endif
>> +#include <thread>
>>
>>  using namespace llvm;
>>
>> @@ -61,7 +62,7 @@ void elf::unlinkAsync(StringRef Path) {
>>    sys::fs::remove(Path);
>>
>>    // close and therefore remove TempPath in background.
>> -  runBackground([=] { ::close(FD); });
>> +  std::thread([=] { ::close(FD); }).detach();
>>  #endif
>>  }
>>
>>
>>
>> _______________________________________________
>> llvm-commits mailing list
>> llvm-commits at lists.llvm.org
>> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits
>>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20171107/d8790d72/attachment.html>