Skip to content

build on CLANG - #1554

Open
cecilia-nallico wants to merge 1 commit into
alibaba:mainfrom
cecilia-nallico:fixing_clang_build
Open

build on CLANG#1554
cecilia-nallico wants to merge 1 commit into
alibaba:mainfrom
cecilia-nallico:fixing_clang_build

Conversation

@cecilia-nallico

@cecilia-nallico cecilia-nallico commented Jul 15, 2026

Copy link
Copy Markdown

Various things have to be updated for a modern clang build to pass.

We're using this command:

cmake -B build -D PHOTON_BUILD_TESTING=ON -DPHOTON_CXX_STANDARD=20 -DCMAKE_C_COMPILER=/usr/bin/clang -DCMAKE_CXX_COMPILER=/usr/bin/clang++
  • some overriding methods were missing the override.
  • we had one too many attribute pop in crc.cpp
  • in some examples and tests we're initializing a char[] from a flag which is forbidden in clang unless the VLA extension is enabled.

@CLAassistant

CLAassistant commented Jul 15, 2026

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@lihuiba

lihuiba commented Jul 15, 2026

Copy link
Copy Markdown
Collaborator

Thanks for your contribution. Please sign the CLA so that we can accept it.

BTW, which version of clang are you using? In Linux?

We are using clang shipped with macOS these years, and it compiles fine. My current version is Apple clang version 21.0.0 (clang-2100.1.1.101).

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