[libc-commits] [PATCH] D74652: [libc] Add support library for use with tests.

[Siva Chandra via Phabricator via libc-commits]

sivachandra added a comment.

Having thought through this patch and its use again, I am currently of the opinion that we should not really be creating equivalents of the library implementations. For example, if we want `abort`, we should use `__llvm_libc::abort`. The `abort` implementation will have its own atomic test and that should be good enough of a confidence to use it in other tests. Likewise, we should be using the `__llvm_libc::thrd_create` and friends instead of using equivalents build over other libraries. We can choose to employ  C++ convenience however - that is, we could implement a thread class using the `__llvm_libc::thrd_*` functions.

WDYT?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D74652