Skip to content

forms: Fix issue with Qt 6.8.3 and Clang 21#1336

Merged
RytoEX merged 1 commit intoobsproject:masterfrom
PatTheMav:clang-21-fix
Mar 30, 2026
Merged

forms: Fix issue with Qt 6.8.3 and Clang 21#1336
RytoEX merged 1 commit intoobsproject:masterfrom
PatTheMav:clang-21-fix

Conversation

@PatTheMav
Copy link
Copy Markdown
Member

Description

Changes include for QMessageBox to use the default umbrella header.

Motivation and Context

Clang 21 enabled built-in support for "__yield" but requires including "arm_acle.h" to work. Qt's "qyieldcpu" header will emit code using the function without including the declaration and thus fail compilation

A fix for Qt is pending, but AppleClang 21 is shipped since 2026-03-24 with Xcode 26.4 and thus the project cannot be compiled after a toolchain update otherwise.

For an unknown reason, using the umbrella header fixes the issue.

How Has This Been Tested?

Tested OS(s): macOS 26 with Xcode 26.4

Types of changes

  • Bug fix (non-breaking change which fixes an issue)

Checklist:

  • I have read the Contributing Guidelines.
  • All commit messages are properly formatted and commits squashed where appropriate.
  • My code is not on master or a release/* branch.
  • The code has been tested.
  • I have included updates to all appropriate documentation.

Clang 21 enabled built-in support for "__yield" but requires including
"arm_acle.h" to work. Qt's "qyieldcpu" header will emit code using the
function without including the declaration and thus fail compilation

A fix for Qt is pending, but AppleClang 21 is shipped since 2026-03-24
with Xcode 26.4 and thus the project cannot be compiled after a
toolchain update otherwise.
@RytoEX RytoEX self-assigned this Mar 27, 2026
@exeldro
Copy link
Copy Markdown
Contributor

exeldro commented Mar 28, 2026

I can confirm this helped fix my Mac build

@RytoEX RytoEX merged commit 89f263f into obsproject:master Mar 30, 2026
1 check passed
@PatTheMav PatTheMav deleted the clang-21-fix branch April 6, 2026 11: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.

3 participants