Skip to content

Fix: remove last VLA in msp432p4.c#2183

Merged
dragonmux merged 1 commit intoblackmagic-debug:mainfrom
ALTracer:fix/alloca-msp432p4
Feb 2, 2026
Merged

Fix: remove last VLA in msp432p4.c#2183
dragonmux merged 1 commit intoblackmagic-debug:mainfrom
ALTracer:fix/alloca-msp432p4

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

Comments