Skip to content

Force-include arm_acle.h on aarch64 macOS to work around Qt qyieldcpu.h - #11

Open
novikov-alexander wants to merge 1 commit into
sonic-visualiser:defaultfrom
novikov-alexander:alnovi/qtYieldBuildFix
Open

novikov-alexander wants to merge 1 commit into
sonic-visualiser:defaultfrom
novikov-alexander:alnovi/qtYieldBuildFix

Conversation

@novikov-alexander

Copy link
Copy Markdown

Qt's qyieldcpu.h calls the ACLE intrinsic __yield() under __has_builtin(__yield) without including <arm_acle.h>. Apple clang 21 reports the builtin as available but makes the resulting implicit declaration an error, so every translation unit reaching <QtCore/qatomic.h> fails to compile.

Qt's qyieldcpu.h calls the ACLE intrinsic __yield() under
__has_builtin(__yield) without including <arm_acle.h>. Apple clang 21
reports the builtin as available but makes the resulting implicit
declaration an error, so every translation unit reaching
<QtCore/qatomic.h> fails to compile.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@novikov-alexander

Copy link
Copy Markdown
Author

Hi @cannam,
I wasn't able to compile the project on the modern MacOS, that is why made this fix.
Please, take a look.

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