This patch adds a document describing the status of the string functions
in LLVM-libc.
Details
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
Thanks for doing this! I do want to pull all of the impllemented/unimplemented functions into a list like the C++ folks have it, but this is nice to have.
It would also be nice to check to see if the _s functions are still being considered for removal and include that information here. http://www.open-std.org/jtc1/sc22/wg14/www/docs/n1967.htm
libc/docs/strings.rst | ||
---|---|---|
34–40 | Outside of Google EXEgesis mostly refers to llvm-exegesis. It is not really connected to string functions (even though it's been developed by the same people). | |
44 | In this document, the Designed and Implemented columns are always equal. Maybe there is more value in telling which architectures are actually implemented ? | |
103 | Maybe this should go into an Additional Functions section? |
addressed comments.
libc/docs/strings.rst | ||
---|---|---|
44 | Most of these string functions are completely platform independent, since they work on individual bytes at a time, so giving all of them architecture specific statuses feels redundant. On the other hand, the memory functions and the conversion functions do have possible platform differences, so tracking them could be helpful. I think the biggest problem is that all of these functions already have versions for all of our supported platforms, so no matter how we slice it the table will just say "Done" everywhere, except for the functions that we haven't started. | |
103 | I like them here because it puts them before the functions that aren't implemented yet. |
libc/docs/strings.rst | ||
---|---|---|
44 | Perhaps a more useful way to present this is to just say whether an implementation is available or not. So, just one column Available under which you say YES if available, nothing if not. You can potentially add a Notes column in future to add additional notes if required. For a reader not involved in the development, that something has been designed or not does not help much. A person interested in contributing can engage on discourse et al to gather more information. |
libc/docs/strings.rst | ||
---|---|---|
44 | Yep in that case a single column is enough. |
Still LGTM with the suggested inline fix.
libc/docs/strings.rst | ||
---|---|---|
85 | s/Designed/Available everywhere? |
address final comments
libc/docs/strings.rst | ||
---|---|---|
85 | I went with Available over Available Everywhere since it allows for specifying which platforms it is available on if that's not everywhere. |
Outside of Google EXEgesis mostly refers to llvm-exegesis. It is not really connected to string functions (even though it's been developed by the same people).
Maybe call the section Primary memory function without description and leave the TODO assigned to me (omitting the exegesis word).