Skip to content

fix clang#28

Open
Firestar99 wants to merge 3 commits intomainfrom
clang
Open

fix clang#28
Firestar99 wants to merge 3 commits intomainfrom
clang

Conversation

@Firestar99
Copy link
Copy Markdown
Member

@Firestar99 Firestar99 commented Apr 1, 2026

  • fix spirv-tools not building with clang due to unknown option -Wno-stringop-overflow
  • add ci testing with clang
  • don't error on warnings by default, to prevent compilers adding warnings in the future from failing builds for end users
  • I tried setting up windows with clang and gcc CI, but setting CCX makes the cc crate think it's still an msvc compiler instead of a clang / gcc compiler, so everything just fails.

@Firestar99 Firestar99 force-pushed the clang branch 4 times, most recently from b66a59f to 72bd683 Compare April 1, 2026 11:17
@Firestar99 Firestar99 changed the title test clang fix clang Apr 1, 2026
@Firestar99 Firestar99 force-pushed the clang branch 2 times, most recently from fe88ebd to 27c35d6 Compare April 1, 2026 11:49
@Firestar99 Firestar99 force-pushed the clang branch 2 times, most recently from 600e447 to b8ac4f5 Compare April 1, 2026 12:10
@Firestar99 Firestar99 marked this pull request as ready for review April 1, 2026 12:29
Comment on lines 121 to -127
if compiler.is_like_gnu() {
build
.flag("-Wall")
.flag("-Wextra")
.flag("-Wnon-virtual-dtor")
.flag("-Wno-missing-field-initializers")
.flag("-Wno-stringop-overflow")
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

If is_like_gnu is telling a lie then i think there's something wrong?

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