Skip to content

feat(quickemu): add CPU flag management with deduplication#1816

Merged
flexiondotorg merged 1 commit intomasterfrom
cpu-flags
Jan 24, 2026
Merged

feat(quickemu): add CPU flag management with deduplication#1816
flexiondotorg merged 1 commit intomasterfrom
cpu-flags

Conversation

@flexiondotorg
Copy link
Member

  • Add CPU_FLAG_MAP and helper functions reset_cpu_flags and add_cpu_flag
  • Validate flag format, deduplicate entries and detect +/- and value conflicts
  • Replace direct CPU string concatenation in configure_cpu with add_cpu_flag calls
  • Centralise CPU flag logic to improve macOS and Windows CPU feature selection

IMPACT: prevents duplicate or conflicting CPU flags being passed to QEMU and
simplifies future CPU feature handling.

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)

Checklist:

  • I have performed a self-review of my code

- Add CPU_FLAG_MAP and helper functions reset_cpu_flags and add_cpu_flag
- Validate flag format, deduplicate entries and detect +/- and value
conflicts
- Replace direct CPU string concatenation in configure_cpu with
add_cpu_flag calls
- Centralise CPU flag logic to improve macOS and Windows CPU feature
selection
IMPACT: prevents duplicate or conflicting CPU flags being passed to QEMU
and
simplifies future CPU feature handling.

Signed-off-by: Martin Wimpress <martin@wimpress.org>
Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

No issues found across 1 file

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.

@flexiondotorg flexiondotorg merged commit aa4211b into master Jan 24, 2026
4 of 5 checks passed
@flexiondotorg flexiondotorg deleted the cpu-flags branch January 24, 2026 19:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant