[libc-commits] [PATCH] D82846: [libc] Add documentation for clang-tidy checks.
Siva Chandra via Phabricator via libc-commits
libc-commits at lists.llvm.org
Mon Jul 6 13:24:11 PDT 2020
sivachandra added inline comments.
================
Comment at: libc/docs/clang_tidy_checks.rst:31
+An advantage of writing a libc in C++ is the ability to use namespaces in our
+implementation. However, implemented entry points must be included within the
+correct namespace in order to export the C symbols properly. Without a formal
----------------
PaulkaToast wrote:
> sivachandra wrote:
> > I think the wording and examples should talk about the general case that all implementation pieces should be under the __llvm_libc namespace.
> Gotcha, let me know if this is better.
I think the note about entrypoints is still inaccurate. Just remove it, and the rest will still be OK.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D82846/new/
https://reviews.llvm.org/D82846
More information about the libc-commits
mailing list