Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Implemented GC methods for StableApiDefinition for TruffleRuby support. #516

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

goyox86
Copy link
Collaborator

@goyox86 goyox86 commented Feb 6, 2025

Solves #447

Implemented:

  • StableApiDefinition::gc_adjust_memory_usage.
  • StableApiDefinition::gc_writebarrier.
  • StableApiDefinition::gc_writebarrier_unprotect.

Note:

I have to tweak the testing macros so it would allow multiple arguments for the data_builder, in the meanwhile I just expanded the macro and tweaked it manually.

@goyox86 goyox86 force-pushed the goyox86/stable-api-gc-methods branch from deb0a59 to cf7c3d1 Compare February 6, 2025 17:19
@goyox86
Copy link
Collaborator Author

goyox86 commented Feb 6, 2025

Looking at Truffle's failure I think they don't have a C API stub for impl_gc_writebarrier, impl_gc_writebarrier_unprotect.

@goyox86 goyox86 requested a review from ianks February 7, 2025 10:12
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant