Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Arm64 macOS doesn't support long double #2175

Open
suborb opened this issue Jan 24, 2023 · 1 comment
Open

Arm64 macOS doesn't support long double #2175

suborb opened this issue Jan 24, 2023 · 1 comment
Labels

Comments

@suborb
Copy link
Member

suborb commented Jan 24, 2023

Which results in a bunch of warnings when using 64 bit integers in sccz80:

rshift.c:525:31: warning: On this host, 64 bit constants may not be correct

This is an apple thing, since floats larger than 64 bits are available on Linux/arm64

We use a long double to hold constant values, falling back to a double means that we're missing 10/11 bits of mantissa precision for a lossless conversion to a uint64_t/int64_t

Given how little 64 bit values are used, it's not massively important, but I should fix it.

@suborb suborb added the sccz80 label Jan 24, 2023
@suborb
Copy link
Member Author

suborb commented Oct 5, 2023

Also related, to recent armhf failures on snap craft:

zcc +test -vn  -I../../framework -DNO_LOG_RUNNING -DNO_LOG_PASSED ../../framework/test.c mult.c -o test_mult.bin  -m
mult.c:37:50: warning: On this host, 64 bit constants may not be correct
 [-Wlimited-range]
mult.c:53:33: warning: On this host, 64 bit constants may not be correct
 [-Wlimited-range]
mult.c:54:33: warning: On this host, 64 bit constants may not be correct
 [-Wlimited-range]
mult.c:56:29: warning: On this host, 64 bit constants may not be correct
 [-Wlimited-range]
mult.c:57:29: warning: On this host, 64 bit constants may not be correct
 [-Wlimited-range]
 06:39:09  Starting test 
Starting suite Multiplication Tests (7 tests)
Running test test_mult_unsigned_longlong.....failed mult.c:53 (3 * 0x2AAAAAAAAAAAAAAB)
7 run, 6 passed, 1 failed

Ticks: 579522
 06:39:09  Test failure 

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant