quickcpplib icon indicating copy to clipboard operation
quickcpplib copied to clipboard

No __int128 on 32 bit platforms

Open clausecker opened this issue 2 years ago • 4 comments

The __int128 type is only supported on 64 bit platforms with gcc and clang. To detect its availability correctly, use

#if defined(__SIZEOF_INT128__)

instead of

#if defined(__GNUC__) || defined(__clang__)

clausecker avatar Oct 24 '23 05:10 clausecker

Thanks for the BR.

I'm seeing in the source code here:

#if(defined(__GNUC__) || defined(__clang__)) && __SIZEOF_INT128__ > 0

i.e. never are we not testing for __SIZEOF_INT128__. Can you say on which line did you find this?

ned14 avatar Oct 26 '23 12:10 ned14

Hm looks like you may have already fixed that, but as you do not make releases, it is unknown how long it takes for these changes to percolate downstream.

clausecker avatar Oct 26 '23 15:10 clausecker

Out of interest: Are you using a package manager? And if yes: Which one?

BurningEnlightenment avatar Oct 27 '23 05:10 BurningEnlightenment

I'm a maintainer of the FreeBSD ports collection where I noticed this problem in the version of quickcpplib we ship.

clausecker avatar Oct 27 '23 05:10 clausecker

I'm going to assume this is fixed and close it. Thanks for the BR!

ned14 avatar Jun 14 '24 09:06 ned14