[PATCH] D54198: [CMake][ASan][HWASan] Set an explicit dependency on libc++ headers

Petr Hosek via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 6 23:49:03 PST 2018


phosek created this revision.
phosek added reviewers: vitalybuka, morehouse.
Herald added subscribers: Sanitizers, llvm-commits, mgorny, kubamracek.
Herald added a reviewer: EricWF.

We have seen failing builds due to a race condition between
RTAsan_dynamic and libc++ headers builds, specifically libc++
headers depend on __config and if this header hasn't been copied
into the final location, including other headers will typically
result in failure. To avoid this race, we add an explicit dependency
on libc++ headers which ensures that they've been copied into place
before the sanitizer object library build starts.


Repository:
  rCRT Compiler Runtime

https://reviews.llvm.org/D54198

Files:
  compiler-rt/lib/asan/CMakeLists.txt
  compiler-rt/lib/hwasan/CMakeLists.txt

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D54198.172900.patch
Type: text/x-patch
Size: 4890 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20181107/57c6cdb1/attachment.bin>


More information about the llvm-commits mailing list