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.

Sign up for GitHub

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

Jump to bottom

replace jl_gc_wb_binding with jl_gc_wb #51854

Merged
merged 1 commit into from
Oct 25, 2023
Merged

replace jl_gc_wb_binding with jl_gc_wb #51854

merged 1 commit into from
Oct 25, 2023

Conversation

d-netto
Copy link
Member

@d-netto d-netto commented Oct 25, 2023 •
edited
Loading

jl_gc_wb_binding has become a thin wrapper around jl_gc_wb now that jl_binding_t is first class.

Replace it with jl_gc_wb to clean-up the write-barrier uses a bit.

@d-netto d-netto requested a review from vtjnash October 25, 2023 03:58
@d-netto d-netto added the GC Garbage collector label Oct 25, 2023
@vtjnash vtjnash added the status:merge me PR is reviewed. Merge when all tests are passing label Oct 25, 2023
@vtjnash vtjnash merged commit 6e5dd59 into master Oct 25, 2023
4 of 8 checks passed
@vtjnash vtjnash deleted the dcn-gc-wb-binding branch October 25, 2023 21:28
@oscardssmith oscardssmith removed the status:merge me PR is reviewed. Merge when all tests are passing label Oct 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@vtjnash vtjnash Awaiting requested review from vtjnash

Assignees
No one assigned
Labels
GC Garbage collector
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@d-netto @vtjnash @oscardssmith