Also, the unused specializations of __int128_t have been removed.
Details
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
libc/src/__support/CPP/Limits.h | ||
---|---|---|
77–88 | Can you just include "Uint128.h" and only specialize the template for UInt128 so that there is no need for #ifdef here? | |
libc/src/__support/CPP/TypeTraits.h | ||
59–66 | Can you just include "Uint128.h" and only specialize the template for UInt128 so that there is no need for #ifdef here? | |
libc/test/src/__support/high_precision_decimal_test.cpp | ||
353–354 | #ifdef's are not needed in this test anymore. | |
libc/test/src/__support/str_to_float_test.cpp | ||
317 | Not needed anymore. | |
libc/utils/UnitTest/CMakeLists.txt | ||
8 | Too many tests using uint128 implicitly I guess? | |
libc/utils/UnitTest/LibcTest.cpp | ||
63–73 | Combine the specializations with UInt128. | |
265–279 | Combine the specializations with UInt128. |
Address comments.
libc/src/__support/CPP/Limits.h | ||
---|---|---|
77–88 | I have added comments now to explain why we need both specializations. Essentially, we need both so that we can uniitest UInt<128> on platfroms where UInt128 resolves to __uint128_t. | |
libc/src/__support/CPP/TypeTraits.h | ||
59–66 | Same reason as above; added comments here as well. | |
libc/utils/UnitTest/CMakeLists.txt | ||
8 | This change isn't changing the situation. The deps were incomplete previously. | |
libc/utils/UnitTest/LibcTest.cpp | ||
63–73 | Same reason as above. Added a comment now. | |
265–279 | Ditto. |
I have landed this now unblock other work. I am happy to do follow up changes as required.
Can you just include "Uint128.h" and only specialize the template for UInt128 so that there is no need for #ifdef here?