libera/#clasp - IRC Chatlog
Search
12:45:53
kpoeck
::notify drmeister: I have trouble with your latest changes to boehm_gc. Building in parallel fails with thread_suspend failed. This message seems to come from here: https://github.com/ivmai/bdwgc/blob/release-8_2/darwin_stop_world.c#L635. Did you have a chance to test the change on macos?
12:54:58
drmeister
kpoeck: Yeah - the new approach was working and I'm running into trouble as well. My errors are different from yours.
12:55:48
drmeister
kpoeck: Are you building the llvm14 branch? I thought I hadn't touched the main branch.
12:55:56
kpoeck
I wonder a bit about the configuration. Woudn't we need the result of platform specific configuration ?
12:59:25
drmeister
What do you think of this change? It appeared harmless and now you are running into a problem.
13:00:44
drmeister
I didn't read through every source file to get an idea of everything that is included. Let me do that first and then we can figure out what to put in mygc.c
13:03:06
drmeister
Now I'm getting a "thread_suspend failed" as well. I swear I didn't see that last night.
13:06:05
drmeister
I'm going to read through each of the files #include'd when I go `#include "extra/gc.c"` and each of the files #include'd by them and so on. Transitively. I'll build a list and then we will have a full list of what is included.
13:06:22
drmeister
Huh - maybe I can get there faster by generating the preprocessed output of compiling mygc.c
13:10:17
drmeister
I should have been more careful before pushing it to main - I never seem to learn. (sigh).
13:11:20
drmeister
I was so excited about it. The thing I liked about MPS was including it all within mygc.c - it gives us total control of the configuration and sort of removes a dependency of a properly configured and built library.
13:14:31
kpoeck
The configuration flags I have are: --disable-parallel-mark --enable-threads=pthreads --enable-large-config --enable-handle-fork=yes --enable-static=yes
13:16:35
drmeister
kpoeck: Ok - you are exactly right. We need a full boehm config.h file. I'm a bit behind you in thinking...
13:17:03
drmeister
./configure in the bdwgc directory builds a config.h file and the way we are pulling it we don't get that.
13:18:11
drmeister
I think your original idea was great - pull a config.h file from building the library.