Skip to content

Conversation

@ALTracer
Copy link
Contributor

@ALTracer ALTracer commented Feb 1, 2026

Detailed description

  • Not a new feature.
  • The existing problem is VLA/alloca left over from PR1606 refactoring, specifically v1.9.0-1188-g f94ed09.
  • This PR completes the replacement (to arrays with known size of 20+33 matching Cortex-M4F).

Found using ag alloca\\\( src/, also matches git grep "alloca(" origin/main -- src/target/*.c. Related: v1.10.0-244-g c0a8672 of PR1661.
Not tested.

Your checklist for this pull request

Closing issues

Copy link
Member

@dragonmux dragonmux left a comment

Choose a reason for hiding this comment

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

LGTM, merging. Thank you for the contribution!

@dragonmux dragonmux merged commit 27144b5 into blackmagic-debug:main Feb 2, 2026
38 of 39 checks passed
@dragonmux dragonmux added this to the v2.1 release milestone Feb 2, 2026
@dragonmux dragonmux added the Enhancement General project improvement label Feb 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Enhancement General project improvement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants