[libc-commits] [PATCH] D76825: [libc] Move implementations of cosf, sinf, sincosf to src/math directory.
Petr Hosek via Phabricator via libc-commits
libc-commits at lists.llvm.org
Thu Apr 16 01:34:01 PDT 2020
phosek accepted this revision.
phosek added a comment.
LGTM, I had the same concern as @abrachet regarding some of the readability bits (curly braces, minimizing variable scopes) but keeping the code as close to original SGTM as well. If we decide to make those changes, we could probably automate them later using a tool like clang-tidy.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D76825/new/
https://reviews.llvm.org/D76825
More information about the libc-commits
mailing list