Skip to content

feat: setup initial oscars integration#5426

Open
shruti2522 wants to merge 3 commits into
boa-dev:dev/oscars-gcfrom
shruti2522:oscars-integration
Open

feat: setup initial oscars integration#5426
shruti2522 wants to merge 3 commits into
boa-dev:dev/oscars-gcfrom
shruti2522:oscars-integration

Conversation

@shruti2522

Copy link
Copy Markdown
Contributor

this is an early draft with the initial setup to integrate oscars in boa. We are currently using null_collector_branded as the baseline for integration

It changes the following:

  • cleaned up boa/core/gc/src/lib.rs by moving the oldlogic into boa_allocator.rs
  • added the oscars_backend feature flag to toggle between the current GC and the new oscars gc

(Note: building with the oscars_backend feature will currently fail since the engine still needs the new 'gc lifetimes to be threaded. I will be adding those updates in the next few commits)

@github-actions github-actions Bot added Waiting On Review Waiting on reviews from the maintainers C-Dependencies Pull requests that update a dependency file C-GC Issue related to garbage collection labels Jul 10, 2026
@github-actions github-actions Bot added this to the v1.0.0 milestone Jul 10, 2026
@shruti2522 shruti2522 marked this pull request as ready for review July 10, 2026 20:00
@shruti2522 shruti2522 requested a review from a team as a code owner July 10, 2026 20:00
Comment thread Cargo.toml Outdated

[patch."https://github.com/boa-dev/oscars.git"]
oscars = { path = "../oscars/oscars" }
oscars_derive = { path = "../oscars/oscars_derive" }

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Might want to remove this for the upstream code, since it would make it a bit hard to test things using the CI (if we need to)

I think you can still configure this using the $HOME/.cargo/config.toml file

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

C-Dependencies Pull requests that update a dependency file C-GC Issue related to garbage collection Waiting On Review Waiting on reviews from the maintainers

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants