[libcxx-commits] [libcxx] [libc++] std::abs support for _BitInt(N) and __int128 (PR #196532)

[Nikolas Klauser via libcxx-commits]

philnik777 wrote:

> > More generally, `std::abs` is specified in a rather weird way where the argument types are explicitly specified in the standard. We notably don't have a `__int128` overload either. Do we actually want to extend here?
> 
> Good question. The __int128 half closes a concrete gap (currently std::abs(__int128_t(-x)) doesn't compile or routes through abs(double) !); the _BitInt half is part of the broader RFC.
> 
> I can split the PR so __int128 lands on its own and the _BitInt part waits for the broader extensions discussion to settle, or skip one of them; I'm unsure what's the best road to be honest.
> 
> What do you think ?

I've thought about this more and I think I'm happy with this direction.

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