Skip to content

Conversation

ngoldbaum
Copy link
Contributor

I noticed there weren't any free-threaded builds for #149.

@ngoldbaum ngoldbaum force-pushed the free-threaded-ci branch 2 times, most recently from 33516bd to 13f1410 Compare August 26, 2025 20:03
@ngoldbaum
Copy link
Contributor Author

I'm confused why there's a complaint about an uninitialized variable, but only on 3.13t under GCC apparently:

test_pythoncapi_compat_cext.c: In function ‘test_long_api’:
test_pythoncapi_compat_cext.c:1439:26: error: ‘digits’ may be used uninitialized [-Werror=maybe-uninitialized]
 1439 |     ((digit *)digits)[0] = 123;
      |     ~~~~~~~~~~~~~~~~~~~~~^~~~~
test_pythoncapi_compat_cext.c:1431:11: note: ‘digits’ was declared here
 1431 |     void *digits;
      |           ^~~~~~

@vstinner do you have any idea what's going wrong?

@ngoldbaum
Copy link
Contributor Author

It looks like the missing error checking was tripping up the compiler, but only on 3.13t. Not sure why, but adding error checking fixes it. This is ready now, I think.

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