Skip to content

Commit 2a702e4

Browse files
author
Raghuveer Devulapalli
committed
Consolidate CI runs to reduce CI runtime
1 parent 357c5e3 commit 2a702e4

File tree

3 files changed

+14
-102
lines changed

3 files changed

+14
-102
lines changed

.github/workflows/c-cpp.yml

Lines changed: 9 additions & 100 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,8 @@ on:
99
permissions: read-all
1010

1111
jobs:
12-
SKL-gcc9:
12+
# All AVX2 code paths
13+
ADL-gcc9:
1314

1415
runs-on: intel-ubuntu-24.04
1516

@@ -37,67 +38,9 @@ jobs:
3738
ninja
3839
3940
- name: Run test suite on SKL
40-
run: sde -skl -- ./builddir/testexe
41-
42-
SKX-gcc10:
43-
44-
runs-on: intel-ubuntu-24.04
45-
46-
steps:
47-
- uses: actions/checkout@b4ffde65f46336ab88eb53be808477a3936bae11 # v4.1.1
48-
49-
- name: Install dependencies
50-
run: |
51-
sudo apt update
52-
sudo apt -y install g++-10 libgtest-dev meson curl git
53-
54-
- name: Install Intel SDE
55-
run: |
56-
curl -o /tmp/sde.tar.xz https://downloadmirror.intel.com/784319/sde-external-9.24.0-2023-07-13-lin.tar.xz
57-
mkdir /tmp/sde && tar -xvf /tmp/sde.tar.xz -C /tmp/sde/
58-
sudo mv /tmp/sde/* /opt/sde && sudo ln -s /opt/sde/sde64 /usr/bin/sde
59-
60-
- name: Build
61-
env:
62-
CXX: g++-10
63-
run: |
64-
make clean
65-
meson setup -Dbuild_tests=true --warnlevel 2 --werror --buildtype release builddir
66-
cd builddir
67-
ninja
68-
69-
- name: Run test suite on SKX
70-
run: sde -skx -- ./builddir/testexe
71-
72-
TGL-gcc11:
73-
74-
runs-on: intel-ubuntu-24.04
75-
76-
steps:
77-
- uses: actions/checkout@b4ffde65f46336ab88eb53be808477a3936bae11 # v4.1.1
78-
79-
- name: Install dependencies
80-
run: |
81-
sudo apt update
82-
sudo apt -y install g++-11 libgtest-dev meson curl git
83-
84-
- name: Install Intel SDE
85-
run: |
86-
curl -o /tmp/sde.tar.xz https://downloadmirror.intel.com/784319/sde-external-9.24.0-2023-07-13-lin.tar.xz
87-
mkdir /tmp/sde && tar -xvf /tmp/sde.tar.xz -C /tmp/sde/
88-
sudo mv /tmp/sde/* /opt/sde && sudo ln -s /opt/sde/sde64 /usr/bin/sde
89-
90-
- name: Build
91-
env:
92-
CXX: g++-11
93-
run: |
94-
make clean
95-
meson setup -Dbuild_tests=true --warnlevel 2 --werror --buildtype release builddir
96-
cd builddir
97-
ninja
98-
- name: Run test suite on TGL
99-
run: sde -tgl -- ./builddir/testexe
41+
run: sde -adl -- ./builddir/testexe
10042

43+
# All AVX-512 code paths
10144
SPR-gcc13:
10245

10346
runs-on: intel-ubuntu-24.04
@@ -135,7 +78,7 @@ jobs:
13578
- name: Run test suite on SPR
13679
run: sde -spr -- ./builddir/testexe
13780

138-
ADL-ASAN-clang18:
81+
ASAN-clang18:
13982

14083
runs-on: intel-ubuntu-24.04
14184

@@ -169,47 +112,13 @@ jobs:
169112
cd builddir
170113
ninja
171114
172-
- name: Run test suite on SPR
115+
- name: Run test suite on ADL
173116
run: sde -adl -- ./builddir/testexe
174117

175-
SPR-ASAN-clang18:
176-
177-
runs-on: intel-ubuntu-24.04
178-
179-
steps:
180-
- uses: actions/checkout@b4ffde65f46336ab88eb53be808477a3936bae11 # v4.1.1
181-
182-
- name: Install dependencies
183-
run: |
184-
sudo apt update
185-
sudo apt -y install clang-18 libomp-18-dev libgtest-dev meson curl git
186-
187-
- name: Install Intel SDE
188-
run: |
189-
curl -o /tmp/sde.tar.xz https://downloadmirror.intel.com/784319/sde-external-9.24.0-2023-07-13-lin.tar.xz
190-
mkdir /tmp/sde && tar -xvf /tmp/sde.tar.xz -C /tmp/sde/
191-
sudo mv /tmp/sde/* /opt/sde && sudo ln -s /opt/sde/sde64 /usr/bin/sde
192-
193-
- name: Build examples
194-
env:
195-
CXX: clang++-18
196-
run: |
197-
cd examples
198-
make all
199-
200-
- name: Build
201-
env:
202-
CXX: clang++-18
203-
run: |
204-
make clean
205-
meson setup -Dbuild_tests=true -Duse_openmp=true -Db_sanitize=address,undefined -Dfatal_sanitizers=true -Dasan_ci_dont_validate=true -Db_lundef=false --warnlevel 0 --buildtype release builddir
206-
cd builddir
207-
ninja
208-
209118
- name: Run test suite on SPR
210119
run: sde -spr -- ./builddir/testexe
211120

212-
SKX-SKL-openmp:
121+
openmp-build:
213122

214123
runs-on: intel-ubuntu-24.04
215124

@@ -236,10 +145,10 @@ jobs:
236145
cd builddir
237146
ninja
238147
239-
- name: Run test suite on SKX and SKL
148+
- name: Run test suite on SKX and ADL
240149
run: |
241150
sde -skx -- ./builddir/testexe
242-
sde -skl -- ./builddir/testexe
151+
sde -adl -- ./builddir/testexe
243152
244153
SPR-gcc13-special-cases:
245154

tests/test-keyvalue.cpp

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,12 +17,14 @@ class simdkvsort : public ::testing::Test {
1717
std::iota(arrsize.begin(), arrsize.end(), 0);
1818
std::iota(arrsize_long.begin(), arrsize_long.end(), 0);
1919
#ifdef XSS_USE_OPENMP
20-
// These extended tests are only needed for the OpenMP logic
20+
// Test larger arrays only when built with openMP
21+
arrsize_long.clear();
2122
arrsize_long.push_back(10'000);
2223
arrsize_long.push_back(100'000);
2324
arrsize_long.push_back(1'000'000);
2425
#endif
2526

27+
2628
arrtype = {"random",
2729
"constant",
2830
"sorted",

tests/test-qsort.cpp

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,8 @@ class simdsort : public ::testing::Test {
1313
std::iota(arrsize.begin(), arrsize.end(), 0);
1414
std::iota(arrsize_long.begin(), arrsize_long.end(), 0);
1515
#ifdef XSS_USE_OPENMP
16-
// These extended tests are only needed for the OpenMP logic
16+
// Test larger arrays only when built with openMP
17+
arrsize_long.clear();
1718
arrsize_long.push_back(10'000);
1819
arrsize_long.push_back(100'000);
1920
arrsize_long.push_back(1'000'000);

0 commit comments

Comments
 (0)