diff options
author | Oliver O'Halloran <oohall@gmail.com> | 2019-11-14 14:36:52 +1100 |
---|---|---|
committer | Oliver O'Halloran <oohall@gmail.com> | 2019-11-14 16:10:21 +1100 |
commit | 7abb0ef549083677bf9ea4121eb19f6e7eb34aef (patch) | |
tree | 4a101dab03888d8397348fc4de71b31fe9dc6519 /opal-ci | |
parent | 2899bda5e7fcf77b69727b03d0ab8acd9d25a29f (diff) | |
download | skiboot-7abb0ef549083677bf9ea4121eb19f6e7eb34aef.zip skiboot-7abb0ef549083677bf9ea4121eb19f6e7eb34aef.tar.gz skiboot-7abb0ef549083677bf9ea4121eb19f6e7eb34aef.tar.bz2 |
opal-ci: Remove clang builds on 18.04
commit 3734172b0f40 ("build: fix non-constant build asserts") added the
following check in asm/head.S:
.if reset_patch_end - reset_patch_start > 0x100
.error "Reset patch overflow"
.endif
Some older versions of clang such as the one that ships with ubuntu
18.04 (vlang 6.0.0) don't consider symbol addresses as constant which
causes the following error:
asm/head.S:578:5: error: expected absolute expression
.if reset_patch_end - reset_patch_start > 0x100
^
asm/head.S:579:2: error: Reset patch overflow
.error "Reset patch overflow"
This appears to be a bug with the clang assembler that is fixed with
versions of clang. Clang support in skiboot is best-effort so removing
the clang builds from the Ubuntu 18.04 CI script is preferable to
working around the limitations of the older versions of clang.
Signed-off-by: Oliver O'Halloran <oohall@gmail.com>
Diffstat (limited to 'opal-ci')
-rwxr-xr-x | opal-ci/build-ubuntu-18.04.sh | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/opal-ci/build-ubuntu-18.04.sh b/opal-ci/build-ubuntu-18.04.sh index 6fd755f..48eb825 100755 --- a/opal-ci/build-ubuntu-18.04.sh +++ b/opal-ci/build-ubuntu-18.04.sh @@ -29,8 +29,3 @@ rm -rf builddir mkdir builddir make SRC=$(pwd) -f ../Makefile -C builddir -j${MAKE_J} make clean - -echo "Building with clang..." -make clean -make -j${MAKE_J} CC=clang -make -j${MAKE_J} CC=clang check |