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

Change arena end ptr #1177

Merged
merged 28 commits into from
Dec 10, 2024
Merged

Change arena end ptr #1177

merged 28 commits into from
Dec 10, 2024

Conversation

stevenmeker
Copy link
Collaborator

@stevenmeker stevenmeker commented Dec 6, 2024 •
edited
Loading

  • arena_end_ptr() now returns char *; changes percolated through collector
  • young_alloc_ptr() and old_alloc_ptr() inlined
  • evacuate() made a member function of class arena

@stevenmeker
use std::swap
2f69066
@stevenmeker
make megabye_malloc() private member function; added data members cur…
fa25f36 
…rent_block_ptr and collection_block_ptr; added initialization for data members
@stevenmeker
make ptr_diff() a simple subtraction
a94b6a9
@stevenmeker
reimplement class arena
259290d
@stevenmeker
deleted decl for megabyte_malloc(), fresh_block(), code cleaning
41a9975
@stevenmeker
added initialize_semispace() to avoid relying on UB for initializatio…
30a0b71 
…n; moved some inline functions from arena.cpp to arena.h; make struct memory_block_header in a union; fix comments
@stevenmeker
keep track of notional number of blocks in each semispace
62a8ee7
@stevenmeker
more accurate simulation of old collection policy; deleted slow_alloc…
3d5ebe8 
…(); fix for move_ptr()
@stevenmeker
used just a pointer addition for move_ptr()
31321ea
@stevenmeker
made move_ptr() inline and added check for end of allocation
f3972a9
@stevenmeker
disable garbage collection to try and isolate bug
d8ef882
@stevenmeker
re-enabled garbage collection - inifinte loop problem has disappeared
4318df9
@stevenmeker
fixed formatting
7a2df99
@stevenmeker
deleted MEM_BLOCK_START; removed corner case from collector
a75b285
@stevenmeker
fix call to initialize_semispace()
e27ec1b
@stevenmeker
deleted dead code youngspace_size() and arena_size()
6af3f6b
@stevenmeker
replaced memory_block_header and associated functionality
bfba1ca
@stevenmeker
fix formatting; clean code and comments
8e31c96
@stevenmeker
fix formatting
9f2e3aa
@stevenmeker
make arena_end_ptr() return char *
98b8404
@rv-jenkins rv-jenkins changed the base branch from master to develop December 6, 2024 02:46
@stevenmeker stevenmeker requested a review from theo25 December 9, 2024 22:55
@dwightguth dwightguth merged commit 0bc565f into develop Dec 10, 2024
13 of 20 checks passed
@dwightguth dwightguth deleted the change_arena_end_ptr branch December 10, 2024 17:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dwightguth dwightguth dwightguth approved these changes

@theo25 theo25 Awaiting requested review from theo25

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@stevenmeker @dwightguth