[PATCH] D71081: gn build: Add support for building scudo and its unit tests.

Peter Collingbourne via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 6 10:23:35 PST 2019


pcc added a comment.

In D71081#1773044 <https://reviews.llvm.org/D71081#1773044>, @pcc wrote:

> Yes, looking through the code as well as the CMake [1], scudo only supports building on Linux, Android and Fuchsia. Most of the code in linux.cpp looks like it would work on non-Linux POSIX, except for the mutex implementation.
>
> It looks like the non-standalone allocator had some level of support for Windows [2] but that wasn't carried over to the standalone allocator.
>
> So we should probably change the blacklist in the supported platforms to a whitelist for now. I'll go ahead and do that.
>
> [1] http://llvm-cs.pcc.me.uk/projects/compiler-rt/cmake/config-ix.cmake#695
>  [2] http://llvm-cs.pcc.me.uk/projects/compiler-rt/lib/scudo/scudo_allocator_secondary.h#110


D71131 <https://reviews.llvm.org/D71131>


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D71081/new/

https://reviews.llvm.org/D71081





More information about the llvm-commits mailing list