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

minor NFC in GC codebase #50991

Merged
merged 1 commit into from
Aug 21, 2023
Merged

minor NFC in GC codebase #50991

merged 1 commit into from
Aug 21, 2023

Conversation

d-netto
Copy link
Member

@d-netto d-netto commented Aug 21, 2023

  • Use GC_PAGE_ALLOCATEDinstead of hardcoded constant
  • Remove stale eytzinger tree comment from gc_mark_outrefs
  • Add some comments to explain why push_lf_page_metadata_back/pop_lf_page_metadata_back can only use a CAS and not be vulnerable to ABA

@brenhinkeller brenhinkeller added the GC Garbage collector label Aug 21, 2023
@vchuravy vchuravy merged commit 74ce6cf into master Aug 21, 2023
7 checks passed
@vchuravy vchuravy deleted the dcn/gc-nfc branch August 21, 2023 13:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
GC Garbage collector
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants