Skip to content

feat(memtrack): add C++ and Rust integration tests #212

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
Merged
not-matthias merged 12 commits into from
Jan 30, 2026
Merged

feat(memtrack): add C++ and Rust integration tests #212

not-matthias merged 12 commits into from
Jan 30, 2026

Conversation

@not-matthias
Copy link
Member

@not-matthias not-matthias commented Jan 23, 2026 •
edited
Loading

  • Add C++ and Rust tests using common allocator functions
  • Also fixes a few issues discovered by the tests:
    • Adds support for previously missing posix_memalign and libstdc++ (libc but for C++)
    • Detect non-default allocators first for binaries
    • Attach to different jemalloc prefixed functions

@not-matthias not-matthias force-pushed the cod-2002-add-rust-c-integration-tests-to-runner branch from 9477d9a to b3231ff Compare January 23, 2026 13:33
@codspeed-hq
Copy link

codspeed-hq bot commented Jan 23, 2026 •
edited
Loading

CodSpeed Performance Report

Merging this PR will not alter performance

Comparing cod-2002-add-rust-c-integration-tests-to-runner (ca2c10c) with main (afa2c89)

Summary

✅ 4 untouched benchmarks

@not-matthias not-matthias force-pushed the cod-2002-add-rust-c-integration-tests-to-runner branch 4 times, most recently from 63327c6 to cea4068 Compare January 23, 2026 16:31
@not-matthias not-matthias force-pushed the cod-2002-add-rust-c-integration-tests-to-runner branch 3 times, most recently from 2670fb3 to 51536d2 Compare January 28, 2026 18:01
GuillaumeLagrange
GuillaumeLagrange approved these changes Jan 29, 2026
View reviewed changes
Copy link
Contributor

@GuillaumeLagrange GuillaumeLagrange left a comment

Choose a reason for hiding this comment

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

olgtm

@not-matthias not-matthias force-pushed the cod-2002-add-rust-c-integration-tests-to-runner branch 5 times, most recently from 57cc5ef to f6a1e68 Compare January 30, 2026 13:41
@not-matthias not-matthias force-pushed the cod-2002-add-rust-c-integration-tests-to-runner branch from f6a1e68 to ca2c10c Compare January 30, 2026 14:09
@not-matthias not-matthias merged commit ca2c10c into main Jan 30, 2026
13 checks passed
@not-matthias not-matthias deleted the cod-2002-add-rust-c-integration-tests-to-runner branch January 30, 2026 14:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@GuillaumeLagrange GuillaumeLagrange GuillaumeLagrange approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@not-matthias @GuillaumeLagrange