Skip to content

[fix][cmake] apple build#562

Merged
BewareMyPower merged 1 commit intoapache:mainfrom
daeho-ro:apple-fix
Apr 1, 2026
Merged

[fix][cmake] apple build#562
BewareMyPower merged 1 commit intoapache:mainfrom
daeho-ro:apple-fix

Conversation

@daeho-ro
Copy link
Copy Markdown
Contributor

Motivation

Build on apple is failed, there is no option -mpclmul for apple target.

clang++: error: unsupported option '-mpclmul' for target 'arm64-apple-darwin25.3.0'

Modifications

Avoid compile option for arm64

Verifying this change

  • Make sure that the change passes the CI checks.

(Please pick either of the following options)

This change is a trivial rework / code cleanup without any test coverage.

(or)

This change is already covered by existing tests, such as (please describe tests).

(or)

This change added tests and can be verified as follows:

(example:)

  • Added integration tests for end-to-end deployment with large payloads (10MB)
  • Extended integration test for recovery after broker failure

Documentation

  • doc-required
    (Your PR needs to update docs and you will update later)

  • doc-not-needed
    (Please explain why) : interface isn't changed, just build config.

  • doc
    (Your PR contains doc changes)

  • doc-complete
    (Docs have been already added)

@lhotari lhotari requested a review from BewareMyPower March 30, 2026 17:03
@BewareMyPower
Copy link
Copy Markdown
Contributor

What's your clang version? I'm using clang++ on arm64 mac as well and it succeeds.

@daeho-ro
Copy link
Copy Markdown
Contributor Author

daeho-ro commented Apr 1, 2026

> clang --version
Apple clang version 21.0.0 (clang-2100.0.123.102)
Target: arm64-apple-darwin25.3.0
Thread model: posix
InstalledDir: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin

@BewareMyPower BewareMyPower merged commit 00c4daa into apache:main Apr 1, 2026
14 checks passed
@BewareMyPower BewareMyPower added this to the 4.2.0 milestone Apr 1, 2026
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.

2 participants