Skip to content

Supports for NodeMemoryPool and Fixed Size Tree Blob#11

Merged
hit9 merged 42 commits into
masterfrom
pool
Apr 28, 2024
Merged

Supports for NodeMemoryPool and Fixed Size Tree Blob#11
hit9 merged 42 commits into
masterfrom
pool

Conversation

@hit9

@hit9 hit9 commented Apr 26, 2024 •
edited
Loading

Copy link
Copy Markdown
Owner

Optional NodePool

  • Split TreeBlob to FixedTreeBlob and Dynamic TreeBlob.
    • FixedTreeBlob tests
    • FixedTreeBlob vs DynamicTreeBlob benchmark on lots of entities
  • A stateful node must export a type alias named Blob
  • size info apis: Size(), TreeSize(), NumNodes() , MaxSizeNode(), MaxSizeNodeBlob()
    • tests

Plan:

// Requires size info, can get from root.maxSizeNodeBlob() and root.NumNodes().
// Memory will be continuous on the entity.
FixedTreeBlob<8, 64> blob;

// No need for size info template parameters, but memory aren't continuous.
DynamicTreeBlob blob;

@hit9 hit9 force-pushed the pool branch 2 times, most recently from 222cfd4 to d1e218e Compare April 27, 2024 06:51
@hit9
Add tests for FixedTreeBlob, strange ha: g++ won't compile without a
654b60f 
template keyword in catch_template_test_macros
hit9 added 19 commits April 27, 2024 15:10
@hit9 hit9 merged commit ae47b2d into master Apr 28, 2024
@hit9 hit9 deleted the pool branch April 28, 2024 11:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@hit9