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?