[libc-commits] [PATCH] D106502: [libc] add option to use SCUDO as the allocator

[Siva Chandra via Phabricator via libc-commits]

sivachandra added a comment.

@cryptoad - This adds a build level dependency across projects.  Do you see any problems which this arrangement? We will also start testing this arrangement via integrations tests on the LLVM libc buildbot workers.



================
Comment at: libc/CMakeLists.txt:76
 
+option(LLVM_LIBC_INCLUDE_SCUDO "Include the SCUDO standalone as the allocator for LLVM libc" OFF)
+if(LLVM_LIBC_INCLUDE_SCUDO)
----------------
cheng.w wrote:
> What about set this option `ON` by default?
Not sure if `ON` or `OFF` is the correct default value while LLVM libc is not complete. When it is complete enough, I think default `ON` makes more sense. That said, if you have a reason to make it `ON` do share here.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D106502