Skip to content

Add Clang CI #2304

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
iomaganaris wants to merge 14 commits into
base: main
Choose a base branch
from
+15 −1
Open

Add Clang CI #2304

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
8740702
Use LLVM 21 compiler for python3.10 and simplify autoopt
iomaganaris Feb 11, 2026
695f34a
Use script provided by llvm to install clang 21
iomaganaris Feb 12, 2026
4950e93
Set the CC/CXX paths explicitely
iomaganaris Feb 12, 2026
32ce678
Merge branch 'main' into add_llvm_ci
iomaganaris Feb 12, 2026
bf1ff37
Install libomp and try to debug the CI issues
iomaganaris Feb 12, 2026
1468d3f
Added OpenMP_ROOT
iomaganaris Feb 12, 2026
5b23d5f
Merge remote-tracking branch 'origin/main' into add_llvm_ci
iomaganaris Feb 12, 2026
fca89f2
Install libomp-dev
iomaganaris Feb 12, 2026
a1034b4
Don't set OpenMP_ROOT
iomaganaris Feb 12, 2026
cedd602
Seems like CI is working. Launch tests in parallel
iomaganaris Feb 12, 2026
ae2cc0c
Add clang configuration on top of the existing ones
iomaganaris Feb 12, 2026
c2476a5
Reorder the included matrix configuration to show nicely in the outpu…
iomaganaris Feb 12, 2026
e11d89a
Disable fast-math to see if the clang ci gets fixed
iomaganaris Feb 19, 2026
9069cbd
Merge branch 'main' into add_llvm_ci
iomaganaris Apr 17, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
14 changes: 14 additions & 0 deletions .github/workflows/general-ci.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,11 @@ jobs:
matrix:
python-version: ['3.10','3.14']
simplify: [0,1,autoopt]
compiler: [gcc]
include:
- python-version: '3.14'
simplify: autoopt
compiler: clang

steps:
- uses: actions/checkout@v6
Expand All @@ -29,6 +34,15 @@ jobs:
uses: actions/setup-python@v6
with:
python-version: ${{ matrix.python-version }}
- name: Setup LLVM
if: matrix.compiler == 'clang'
run: |
wget https://apt.llvm.org/llvm.sh
chmod +x llvm.sh
sudo ./llvm.sh 21
Copy link
Copy Markdown
Contributor Author

@iomaganaris iomaganaris Feb 16, 2026

Choose a reason for hiding this comment

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

Suggested change
sudo ./llvm.sh 21
sudo ./llvm.sh

Not sure if we should pinpoint this to a specific version or keep this open to get the last one always

sudo apt-get install -y libomp-dev
echo "CXX=$(which clang++)" >> $GITHUB_ENV
echo "CC=$(which clang)" >> $GITHUB_ENV
- name: Install dependencies
run: |
# Make dependency setup faster
Expand Down
2 changes: 1 addition & 1 deletion dace/config_schema.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -268,7 +268,7 @@ required:
type: str
title: Arguments
description: Compiler argument flags
default: '-fPIC -Wall -Wextra -O3 -march=native -ffast-math -Wno-unused-parameter -Wno-unused-label'
Copy link
Copy Markdown
Contributor Author

@iomaganaris iomaganaris Feb 19, 2026

Choose a reason for hiding this comment

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

This needs to be reverted however it didn't fix all the errors in the Clang CI

default: '-fPIC -Wall -Wextra -O3 -march=native -Wno-unused-parameter -Wno-unused-label'
default_Windows: '/O2 /fp:fast /arch:AVX2 /D_USRDLL /D_WINDLL /D__restrict__=__restrict'

libs:
Expand Down
Loading