-
Notifications
You must be signed in to change notification settings - Fork 909
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
Compiler crash #3874
Comments
@rockwotj which version of TinyGo is this? |
Sorry it was v0.28.1 |
Built from source using the commands to get a statically linked binary |
We see tinygo segfaults and in an effort to reduce noise in CI, we can just retry the build up to 3 times if it fails. Upstream issue: tinygo-org/tinygo#3874 Signed-off-by: Tyler Rockwood <[email protected]>
We see tinygo segfaults and in an effort to reduce noise in CI, we can just retry the build up to 3 times if it fails. Upstream issue: tinygo-org/tinygo#3874 Signed-off-by: Tyler Rockwood <[email protected]>
I'm having the same issue. It seems to happen a few times in a row and then disappear. tinygo version produces:
|
Are you perhaps compiling code that contains generics? I'm aware of a race condition that sometimes happens and I keep postponing to fix, see #3532. |
Yes my code does have generics |
Oh sorry wait no this version does not have any generics. I added generics after we started seeing this crash |
This usually works by chance, but leads to crashes. So we should never ever do this. I'm pretty sure this is the crash behind this issue: #3894 It may also have caused this crash: #3874 I have a suspicion this is also behind the rather crash-prone CircleCI jobs, that we haven't been able to find the source of. But we'll find out soon enough once this fix is merged. To avoid hitting this issue again in the future, I've created a PR to remove these dangerous functions altogether from the go-llvm API: tinygo-org/go-llvm#54
This might fix the crash: #3917 |
This usually works by chance, but leads to crashes. So we should never ever do this. I'm pretty sure this is the crash behind this issue: #3894 It may also have caused this crash: #3874 I have a suspicion this is also behind the rather crash-prone CircleCI jobs, that we haven't been able to find the source of. But we'll find out soon enough once this fix is merged. To avoid hitting this issue again in the future, I've created a PR to remove these dangerous functions altogether from the go-llvm API: tinygo-org/go-llvm#54
This is addressed as part of |
@deadprogram @aykevl I've confirmed that upgrading to v0.30.0 does resolve the issue (I haven't seen any crashes) thanks for your work on this! |
We had a compiler crash during a CI run, however it's the only time this has happened, so maybe the stacktrace is helpful, but note this is likely some race condition somewhere, as this generally doesn't happen.
The text was updated successfully, but these errors were encountered: