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

use 64bit find-and-clear avoiding build failure with GCC14/i386 #439

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

hillu
Copy link

@hillu hillu commented Dec 30, 2024

Rebuilding the Debian package with GCC 14 for the i386 architecture exposes an "incompatible pointer" error, cf. https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1091572

limex_exceptional.h defines a FIND_AND_CLEAR_FN macro that is used on CHUNK_T pointers in most cases, except in

u32 bit = FIND_AND_CLEAR_FN(&word);
I'm pretty sure that I should not change word from an u64a to a CHUNK_T at this point but rather use the 64bit-specific find-and-clear function as done in the patch.

This fixes the build failure, but I am not sure if this is correct. Please advise on how to test this. Are there any test cases fthat stress this specific code?

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

Successfully merging this pull request may close these issues.

1 participant