[libcxx-commits] [PATCH] D94909: [VE] Define FUTEX values

Louis Dionne via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Mon Jan 18 08:56:18 PST 2021


ldionne requested changes to this revision.
ldionne added a comment.
This revision now requires changes to proceed.

Thanks for your contribution. However, it would be useful to have more information before going forward. The goal is to avoid adding random bits of complexity to the library for a system that isn't officially supported and maintained.

- What is VEOS?
- Is this part of an effort to port libc++ to that OS?
- What parts of the library are working on the OS and what parts are not working? Do you plan to subset the library using knobs like `LIBCXX_ENABLE_LOCALIZATION=OFF`?
- Are you able to provide CI for that OS?

Generally speaking, more information about what prompts this patch is welcome.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D94909



More information about the libcxx-commits mailing list