Skip to content

GC test polishing#561

Merged
jmid merged 6 commits into
mainfrom
gc-adjustments
Jun 20, 2025
Merged

GC test polishing#561
jmid merged 6 commits into
mainfrom
gc-adjustments

Conversation

@jmid

@jmid jmid commented Jun 19, 2025

Copy link
Copy Markdown
Collaborator

Analogous to #558 I also had another look at the Gc tests and spotted a couple of things there.
This PR is the result.

  • The biggest change is presentational, where the generator pair is split and instead defined one after another.
  • Second it replaces minor_heap_size_gen with a bit-shifting version rather than "choose from list literal".
  • Third it generates Counters as part of alloc_cmds (like the other observational Gc functions) for consistency.

Finally there are a few smaller meaning preserving code clean ups.

@jmid jmid changed the title GC test revision GC test polishing Jun 20, 2025
@jmid

jmid commented Jun 20, 2025

Copy link
Copy Markdown
Collaborator Author

CI summary: all 50 workflows passed

@jmid jmid merged commit 70ed61b into main Jun 20, 2025
50 checks passed
@jmid jmid deleted the gc-adjustments branch June 20, 2025 06:36
@jmid

jmid commented Jun 20, 2025

Copy link
Copy Markdown
Collaborator Author

CI summary for merge to main: all 51 workflows passed

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant