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

Add a -O3 test file to trigger segfault #4139

Closed
wants to merge 2 commits into from
Closed

Add a -O3 test file to trigger segfault #4139

wants to merge 2 commits into from

Conversation

maurolacy
Copy link

Related to #4136.

@MaxGraey
Copy link
Contributor

@maurolacy As first time contributor your PR required some approval for CI workflow. You have to wait until one of the meinteiners does it.

@maurolacy
Copy link
Author

maurolacy commented Sep 10, 2021

OK. It turns out I couldn't reproduce this in main. It only fails in the version_101 tag (under alpine). So, it seems this is already a solved issue.

Will add a couple of scripts and instructions with the path for reproduction.

Question would be then: when version_102 is going to be released?

@maurolacy
Copy link
Author

maurolacy commented Sep 10, 2021

Bug triggers when building statically on alpine (build script added for reference), but only in the version_101 tag.

If --debug is added, segfault disappears. ?-) Dangling pointers having fun I guess.

If I build a static binary based on the alpine build, but in debian, wasm-opt segfaults when launched without args. More dangling pointers.

Anyway, it seems this is solved / couldn't reproduce in the main branch. Close and wait for 102 I guess?

@kripken
Copy link
Member

kripken commented Sep 10, 2021

Thanks for confirming. Yes, I think we can just release 102 now, I'll open a PR.

@kripken kripken closed this Sep 10, 2021
@kripken kripken mentioned this pull request Sep 10, 2021
@kripken
Copy link
Member

kripken commented Sep 10, 2021

Ok, 102 was tagged and CI is working on builds.

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.

None yet

3 participants