Refactor src/math/hypotf.cpp and test/src/math/hypotf_test.cpp and reuse them for hypot and hypot_test
Details
Diff Detail
- Repository
- rG LLVM Github Monorepo
Unit Tests
Event Timeline
libc/utils/FPUtil/Hypot.h | ||
---|---|---|
121 | In other functions, we have not used the _t suffixes. To be consistent, can you follow the same convention here? If you feel strongly about the _t style, you can do a follow up cleanup pass changing everything to follow the _t style. | |
122 | Same here: other places do not have the _t but use UIntType with the template type. | |
123 | Same. |
libc/utils/FPUtil/Hypot.h | ||
---|---|---|
121 | Changed to UIntType |
clang-tidy: warning: invalid case style for parameter 'mant' [readability-identifier-naming]
not useful
clang-tidy: warning: invalid case style for parameter 'shift_length' [readability-identifier-naming]
not useful