-
Notifications
You must be signed in to change notification settings - Fork 917
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
compile failed with undefined: C._Bool
#3926
Comments
Well this led me down a rather deep rabbit hole to improve CGo compatibility: #3927 I'll look at the actual bug next. |
This fixes #3926. While working on this I've found another bug: if C.bool is referenced from within Go, it isn't available anymore on the C side. This is an existing bug that also applies to float and double, but may be less likely to be triggered there. This bug is something to be fixed at a later time (it has something to do with preprocessor defines).
Here is a fix: #3928 |
#3928 is merged |
Reopening and tagging to close on next release. |
I build dev branch on my own computer. I tested
It's ok. However I tested fermyon/spin#1794 example but building painc.
Is it another issue? |
@aykevl Any idea? |
This fixes tinygo-org#3926. While working on this I've found another bug: if C.bool is referenced from within Go, it isn't available anymore on the C side. This is an existing bug that also applies to float and double, but may be less likely to be triggered there. This bug is something to be fixed at a later time (it has something to do with preprocessor defines).
@Patrick0308 my guess is that they're shipping their own TinyGo variant, that is still the latest release (and not the dev branch). |
Environment:
tinygo version: 0.30.0
operating System: macOS
architecture: aarch64
Description:
Compilation brakes with an undefined C._Bool
Steps to reproduce:
Given the following Go-code:
Run
tinygo run main.go
Expected behavior:
Actual behavior:
The text was updated successfully, but these errors were encountered: