[lld] Fix elf::unlinkAsync detached thread
authorVitaly Buka <vitalybuka@google.com>
Thu, 14 Feb 2019 23:29:28 +0000 (23:29 +0000)
committerVitaly Buka <vitalybuka@google.com>
Thu, 14 Feb 2019 23:29:28 +0000 (23:29 +0000)
commit2694810153cf992823eb45253d26b8567424438f
tree65bca86db4f19b44b67cf4ddebe1dc8c94c5c9ef
parent0b2dca9f830211a1216aa64cb48359af747faac5
[lld] Fix elf::unlinkAsync detached thread

Summary:
So this patch just make sure that the thread is at least stated
before we return from main.

If we just detach then the thread may be actually be stated just after
the process returned from main and it's calling atexit handers. Then the thread may try to create own function static variable and it will
add new at exit handlers confusing libc.

GLIBC before 2.27 had race in that case which corrupted atexit handlers
list. Support for this use-case for other implementation is also unclear,
so we can try just avoid that.

PR40162

Reviewers: ruiu, espindola

Subscribers: emaste, arichardson, jfb, llvm-commits

Tags: #llvm

Differential Revision: https://reviews.llvm.org/D58246

llvm-svn: 354078
lld/ELF/Filesystem.cpp