Refactor src/math/hypotf.cpp and test/src/math/hypotf_test.cpp and reuse them for hypot and hypot_test
Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
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 |
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.