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.