Skip to content

Add 32-bit Linux CI coverage#2585

Open
danyalahmed1995 wants to merge 3 commits into
KhronosGroup:mainfrom
danyalahmed1995:ci/add-32bit-linux
Open

Add 32-bit Linux CI coverage#2585
danyalahmed1995 wants to merge 3 commits into
KhronosGroup:mainfrom
danyalahmed1995:ci/add-32bit-linux

Conversation

@danyalahmed1995
Copy link
Copy Markdown

Adds a 32-bit Ubuntu CI matrix entry using gcc/g++ multilib and -m32 flags.

The sample build is skipped for this cross-compilation entry to avoid requiring 32-bit graphics/system libraries, while header generation and the unit test builds still run across the supported C++ standards.

This covers the Linux 32-bit CI case discussed in #2584 and helps exercise the kind of build configuration involved in #2583.

Tested on my fork:

  • CI Build / ubuntu-24.04 x86, Ninja, g++-14 passed
  • Full fork PR checks passed

Refs #2584.

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Jun 2, 2026

CLA assistant check
All committers have signed the CLA.

Copy link
Copy Markdown
Contributor

@M2-TE M2-TE left a comment

Choose a reason for hiding this comment

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

Good addition, thanks for the contribution!
Just got a few comments regarding the addition of new options in the matrix entry.

Comment thread .github/workflows/build.yml Outdated
Comment thread .github/workflows/build.yml
Comment thread .github/workflows/build.yml Outdated
Comment thread .github/workflows/build.yml Outdated
@sharadhr
Copy link
Copy Markdown
Contributor

sharadhr commented Jun 2, 2026

I would also recommend using Clang with --target=i686-linux-gnu.

Co-authored-by: Jan Kuhlmann <33833587+M2-TE@users.noreply.github.com>
@danyalahmed1995
Copy link
Copy Markdown
Author

@sharadhr I went with gcc/g++ plus -m32 because it matched the existing Ubuntu GCC matrix entries closely and kept the first 32-bit Linux job minimal. It also passed in my fork before opening the PR.

But if you want the CI entry to model cross-compilation more explicitly, I can switch it to Clang with --target=i686-linux-gnu.

Co-authored-by: Jan Kuhlmann <33833587+M2-TE@users.noreply.github.com>
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.

4 participants