[PATCH] D62088: [compiler-rt][builtins] Scaled Integer log10()
Sterling Augustine via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu May 30 13:43:21 PDT 2019
saugustine added a comment.
I can't speak to how useful this would be and for whom, but I'm not opposed to including it if Fuschia has a use for it.
Typically, compiler-support libraryfunctions are all lowercase, and just use a single letter for each type--"i" for integer--prefixed by a "size of type" character: 's' for "single", 'd' for double length as in this example (it's a little archaic), I don't know of a common one for "scaled" though.
Then each one is also numbered. The exact mechanism to choose the number escapes me at the moment, but I'm consulting with someone who should know.
So this should probably be named something more like __log10scaleddiX, while I look for what X should be.
Also, has this been run through clang-format? I can never remember the rules on line-length.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D62088/new/
https://reviews.llvm.org/D62088
More information about the llvm-commits
mailing list