[libc-commits] [libc] [libc] Allow BigInt class to use base word types other than uint64_t. (PR #81634)

Guillaume Chatelet via libc-commits libc-commits at lists.llvm.org
Wed Feb 14 02:19:53 PST 2024


gchatelet wrote:

> > I haven't read the patch yet but just to make sure we're on the same page I'm cross posting this here [#81267 (comment)](https://github.com/llvm/llvm-project/pull/81267#issuecomment-1941301356)
> 
> Thanks, that patch would definitely help with initializing `UInt` classes. Hopefully there won't be any issue merging this and #81267

I synced #81267 and unfortunately there is one issue
```
libc/src/__support/integer_literals.h:122:28: error: class template partial specialization contains a template parameter that cannot be deduced; this partial specialization will never be used [-Wunusable-partial-specialization]
template <size_t N> struct Parser<LIBC_NAMESPACE::cpp::UInt<N>> {
                           ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
libc/src/__support/integer_literals.h:122:18: note: non-deducible template parameter 'N'
template <size_t N> struct Parser<LIBC_NAMESPACE::cpp::UInt<N>> {
```
I need to think this through.

https://github.com/llvm/llvm-project/pull/81634


More information about the libc-commits mailing list