- The Python header generator has been removed.
- Docs giving a highlevel overview of the header gen scheme have been
added.
Differential D70197
[libc] Add a TableGen based header generator. sivachandra on Nov 13 2019, 11:37 AM. Authored by
Details
Diff Detail
Event Timeline
Comment Actions Not of huge importance, more of curiosity, but could you do what you did with a past revision where you pushed a review with an svg file to your fork so we could see what it looks like on your GitHub? Comment Actions Address comments.
Comment Actions It looks quite nice. I've said this before I believe, but I really like the focus on documentation. Its pretty crucial for what were trying to accomplish with the modularity of this libc and the changes that brings to the build process and I think its great so far.
Comment Actions I think this is in a good spot, it's easy to read and modify later as needs change. I don't know much about TableGen, I've tried my best to look through, but it would be preferable if other reviewers would be able to take a look too. Up to your discretion how long you would like to wait to let others chime in. LGTM I think that incorporating clang-format down the line in the header generation process isn't a bad idea. For one it would get rid of that strange white space (if we can't find the cause) and would also get rid of inconsistencies of the placement of *, char * strtok(char *__restrict, const char *__restrict);. Certainly it doesn't matter for now, just something to think about. Also of note, I find a lot of libc's decide to put #pragma GCC system_header. Do you have thoughts on this?
|