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

Build fails with SEXP_USE_MALLOC #809

Merged
merged 1 commit into from
Jan 9, 2022
Merged

Conversation

dpapavas
Copy link
Contributor

@dpapavas dpapavas commented Jan 9, 2022

This is really an issue, not a PR. It's not something I particularly need, but during the investigation of a bug I suspected to be due to the GC, I tried defining SEXP_USE_MALLOC, to see whether the behavior would change. This resulted in a couple of C compile errors, which I fixed, but building just fails a bit later when building Scheme library stubs. I didn't pursue the failure further, but I though I'd report it.

(Unless I'm missing something, this seems to be broken for quite a while - at least since 0.8, so if it has gone undetected until now, perhaps doing away with it altogether would be appropriate.)

@ashinn
Copy link
Owner

ashinn commented Jan 9, 2022

This option is a hack which isn't suitable to any non-trivial program, so I don't guarantee support for it.

If you're having trouble with GC I recommend SEXP_USE_BOEHM or SEXP_USE_CONSERVATIVE_GC at a first pass. There are other GC options to investigate further.

@ashinn ashinn merged commit f126c47 into ashinn:master Jan 9, 2022
@dpapavas
Copy link
Contributor Author

Thanks for the tips. I had tried to SEXP_USE_CONSERVATIVE_GC and it did indeed make the problem "go away", but I had already found out by then, that it was due to not preserving some values I should have been preserving (I suppose, because what needs to be preserved and what doesn't, isn't entirely clear).

I noticed you merged the PR. Does the source build for you correctly with it? As I mentioned, it only fixes the C side of the problem for me; I still get Scheme errors when the FFI processes certain library stubs.

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.

2 participants