Skip to content

Call initialize_collection before enabling GC#44

Merged
qinsoon merged 1 commit intommtk:v1.9.2+RAIfrom
qinsoon:init-collection-before-enable-v1.9.2
Apr 24, 2024
Merged

Call initialize_collection before enabling GC#44
qinsoon merged 1 commit intommtk:v1.9.2+RAIfrom
qinsoon:init-collection-before-enable-v1.9.2

Conversation

@qinsoon
Copy link
Copy Markdown
Member

@qinsoon qinsoon commented Mar 26, 2024

This PR fixes mmtk/mmtk-core#1096. We should call initialize_collection before enabling GC. Related discussion here: #39 (comment)

Copy link
Copy Markdown

@udesou udesou left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@qinsoon qinsoon merged commit d0a47eb into mmtk:v1.9.2+RAI Apr 24, 2024
qinsoon added a commit to qinsoon/julia that referenced this pull request May 6, 2024
qinsoon added a commit that referenced this pull request May 10, 2024
* Call initialize_collection before enabling GC (#44)
* Fix build with stock GC: mmtk_pin_object is conditionaly compiled (#42)
qinsoon pushed a commit that referenced this pull request Mar 14, 2026
…55 (JuliaLang#60505)

Stdlib: NetworkOptions
URL: https://github.com/JuliaLang/NetworkOptions.jl.git
Stdlib branch: master
Julia branch: backports-release-1.12
Old commit: 532992f
New commit: 7034c55
Julia version: 1.12.3
NetworkOptions version: 1.3.0 (Does not match)
Bump invoked by: @fingolfin
Powered by:
[BumpStdlibs.jl](https://github.com/JuliaLang/BumpStdlibs.jl)

Diff:
JuliaLang/NetworkOptions.jl@532992f...7034c55

```
$ git log --oneline 532992f..7034c55
7034c55 Bump actions/checkout from 4 to 5 (#44)
46e14ef Merge pull request #43 from lgoettgens/lg/JET-ca_roots-path_convert
```

Co-authored-by: fingolfin <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants