Skip to content

Fix consensus_tree segfault on arm64 by implementing dynamic stack/heap allocation #243

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

Open
Copilot wants to merge 15 commits into
base: main
Choose a base branch
from
Open

Fix consensus_tree segfault on arm64 by implementing dynamic stack/heap allocation #243

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
7c2f0d5
Initial plan
Copilot Feb 4, 2026
3f731b8
Implement dynamic allocation for ClusterTable and consensus_tree
Copilot Feb 4, 2026
b5ad17d
Fix error message formatting and add sstream include
Copilot Feb 4, 2026
f8cb83b
Address code review feedback: improve comments and add assertion
Copilot Feb 4, 2026
21d044b
Final comprehensive testing - all tests pass
Copilot Feb 4, 2026
529b365
Delete _codeql_detected_source_root
ms609 Feb 4, 2026
3f1bcc9
nEdges msg
ms609 Feb 4, 2026
c699052
rm comments
ms609 Feb 4, 2026
e3cb5c2
Use int32 & char, not int16 & bool
ms609 Feb 4, 2026
4627540
Guard against stack overflow
ms609 Feb 4, 2026
22aecf1
int32 in splits_to_edge
ms609 Feb 4, 2026
ca26ee4
Extra ASSERTions
ms609 Feb 4, 2026
bf2de6c
Support >13k tips
ms609 Feb 4, 2026
5327217
Skip on CRAN
ms609 Feb 4, 2026
ec33427
Change constexpr to const for external package compatibility
Copilot Feb 4, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Loading
Loading