-
Notifications
You must be signed in to change notification settings - Fork 268
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
What about documentation? #172
Comments
Thx for the issue. I agree that more documentation would be appreciated, especially in the headers. Now, I'm not too fond of Doxygen in this case.
I understand the need but I don't have a good alternative apart from structuring the code in a way that makes it obvious what we support. We could definitely improve on that front. Let me know what you think. |
Ok, what about creating a documentation site like googletest with guides, changelog, references for contributors. I created simple docs using VuePress cpu_features_test_docs, implementation link: test_branch |
I also agree that the document is required |
gentle ping @gchatelet |
It seems to me for the usual user, it will be unclear what CPU features have(excluding what was described in README samples).
I would like to propose to introduce documentation that will include the API documentation e.g. Doxygen with GitHub pages or Github Wiki.
Description of structures, what they contain, what each field defined.
Also, it will be great to make it clear what is supported. For example:
AMD CPUID supports:
The text was updated successfully, but these errors were encountered: