[libc-commits] [PATCH] D138608: [libc] Add initial support for a libc implementation for the GPU
Siva Chandra via Phabricator via libc-commits
libc-commits at lists.llvm.org
Sun Nov 27 23:43:38 PST 2022
sivachandra added a comment.
Thanks for the patch. The mechanical aspects of it LGTM but I have a few administrative questions and points to make:
1. You should add a doc in the `docs` directory describing what exactly is a GPU build, its status, how to build and and how to test. You don't have to cram all of this in this patch, but I would like to hear that you do intend to follow up with appropriate documentation.
2. What is the CI story for the GPU build? Everything, from who will own and maintain the CI builders, to the action that is to be taken if a patch breaks the GPU CI builders. Again, you don't have to stand up a GPU builders right away. You can choose to include the plan in the above documentation and work separately in setting up the buidlers.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D138608/new/
https://reviews.llvm.org/D138608
More information about the libc-commits
mailing list