[PATCH] D62245: [compiler-rt] Update comments in asan header to doxygen format; NFC.
Johannes Doerfert via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Sun Jun 2 11:42:11 PDT 2019
jdoerfert accepted this revision.
jdoerfert added a comment.
This revision is now accepted and ready to land.
I think adding all these comments and updating the format is a great thing!
Minor comments to consider:
- I thought we used backslash instead of @ now but I might be wrong. The coding standard uses backslashs, if compiler-rt is not using @ all over the place you should change it.
- Mention in the commit message that you also add descriptions to undocumented functions.
- Mention in the commit message that you improved spelling, grammar, and formatting as well.
I'm not a compiler-rt person but if there is no objection in the next few days, you should go ahead (LGTM).
Repository:
rCRT Compiler Runtime
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D62245/new/
https://reviews.llvm.org/D62245
More information about the llvm-commits
mailing list