Skip to content

Commit 59d39a2

Browse files
committed
ICU-22873 Update to clang18 and enable -Werror
See unicode-org#3134
1 parent f19e598 commit 59d39a2

File tree

1 file changed

+11
-13
lines changed

1 file changed

+11
-13
lines changed

.github/workflows/icu4c.yml

Lines changed: 11 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -314,8 +314,8 @@ jobs:
314314
run: |
315315
cd icu4c/source && ./runConfigureICU --enable-debug --disable-release Linux/clang && make -j -l4.5 check
316316
317-
# Clang Linux 16 with CPP20 and treat warnings as errors
318-
clang16-cpp20-warning-as-errors:
317+
# Clang Linux 18 with CPP20 and treat warnings as errors
318+
clang18-cpp20-warning-as-errors:
319319
runs-on: ubuntu-latest
320320
strategy:
321321
fail-fast: false
@@ -325,21 +325,19 @@ jobs:
325325
- '-std=c++20 -stdlib=libc++'
326326
steps:
327327
- uses: actions/checkout@b4ffde65f46336ab88eb53be808477a3936bae11 # v4.1.1
328-
- name: Install Clang-16
328+
- name: Install Clang-18
329329
run: |
330-
curl -Ls https://apt.llvm.org/llvm-snapshot.gpg.key | sudo apt-key add -
331-
sudo apt-add-repository 'deb http://apt.llvm.org/focal/ llvm-toolchain-focal-16 main'
332-
sudo apt update
333-
sudo apt install -qy clang-16 lld-16 libc++-16-dev libc++abi-16-dev
334-
- name: Clang-16 build and Test
330+
wget https://apt.llvm.org/llvm.sh
331+
chmod u+x llvm.sh
332+
sudo ./llvm.sh 18 all
333+
- name: Clang-18 build and Test
335334
env:
336-
CC: clang-16
337-
CXX: clang++-16
338-
# TODO: add '-Werror' after fixing ICU-22873
339-
CPPFLAGS: '-Wall -Wextra -Wctad-maybe-unsupported'
335+
CC: clang-18
336+
CXX: clang++-18
337+
CPPFLAGS: '-Wall -Wextra -Wctad-maybe-unsupported -Werror'
340338
CXXFLAGS: ${{ matrix.flags }}
341339
run: |
342-
cd icu4c/source && ./runConfigureICU --enable-debug --disable-release Linux/clang && make -j -l4.5 check
340+
cd icu4c/source && ./runConfigureICU --enable-debug --disable-release Linux && make -j -l4.5 check
343341
344342
# MacOS with clang
345343
macos-clang:

0 commit comments

Comments
 (0)