Skip to content

Commit 94f81fc

Browse files
committed
CI: Check it builds with assertions disabled.
Will prevent this kind of mishap: ykjit/yk#881 (comment)
1 parent a02f47d commit 94f81fc

File tree

1 file changed

+21
-0
lines changed

1 file changed

+21
-0
lines changed

.buildbot.sh

Lines changed: 21 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -51,3 +51,24 @@ cmake --build build --target check-all
5151
# ../test-suite
5252
#make -j `nproc`
5353
#llvm-lit -v -j 1 -o results.json .
54+
55+
# Check ykllvm builds with assertions disabled.
56+
#
57+
# In the past things like `Value::dump()` have slipped in to the repo and these
58+
# are not available if LLVM is built without assertions (you'd get a linker
59+
# error).
60+
rm -rf build
61+
mkdir build
62+
cd build
63+
cmake -DCMAKE_INSTALL_PREFIX=${INST_DIR} \
64+
-DLLVM_INSTALL_UTILS=ON \
65+
-DCMAKE_BUILD_TYPE=release \
66+
-DLLVM_ENABLE_ASSERTIONS=OFF \
67+
-DLLVM_ENABLE_PROJECTS="lld;clang" \
68+
-DCLANG_DEFAULT_PIE_ON_LINUX=OFF \
69+
-DBUILD_SHARED_LIBS=ON \
70+
-DCMAKE_C_COMPILER=/usr/bin/clang \
71+
-DCMAKE_CXX_COMPILER=/usr/bin/clang++ \
72+
-GNinja \
73+
../llvm
74+
cmake --build .

0 commit comments

Comments
 (0)