Opened 7 months ago
Closed 7 months ago
#22859 closed enhancement (fixed)
llvm/libclc-22.1.0
| Reported by: | Bruce Dubbs | Owned by: | Xi Ruoyao |
|---|---|---|---|
| Priority: | normal | Milestone: | 13.1 |
| Component: | BOOK | Version: | git |
| Severity: | medium | Keywords: | |
| Cc: |
Description
New major version.
Change History (11)
comment:2 by , 7 months ago
rustc needs a patch to resolve some test failures: https://github.com/rust-lang/rust/commit/bf3ac98d6930ba4e258cf33240c2fe7c99d19eae.patch (though it seems we are not compiling rust to AMDGPU shader so maybe we can simply document the failures instead).
comment:3 by , 7 months ago
spirv-llvm-translator needs:
- https://github.com/KhronosGroup/SPIRV-LLVM-Translator/commit/8e829705b660a4c3a1fcd97885c35b8d78941276.patch
- https://github.com/KhronosGroup/SPIRV-LLVM-Translator/commit/8750bce43eeba2ac54428d095fb46014a7f110e8.patch
- https://github.com/KhronosGroup/SPIRV-LLVM-Translator/commit/daba8b217bc266806ac00095262d1af0ba2ee610.patch
and -D BASE_LLVM_VERSION=22.1.0. Maybe it's easier to wait for a 22 release.
comment:5 by , 7 months ago
| Owner: | changed from to |
|---|---|
| Status: | new → assigned |
comment:6 by , 7 months ago
comment:7 by , 7 months ago
Firefox FTBFS. The bundled rust-bindgen needs a fix: https://patch-diff.githubusercontent.com/raw/rust-lang/rust-bindgen/pull/3278.patch (the only relevant part is the hunk for clang.rs).
comment:8 by , 7 months ago
And it's still not enough. The next error:
5:38.23 In file included from /home/xry111/sources/12.5/firefox-140.8.0/gfx/skia/skia/modules/skcms/src/skcms_TransformHsw.cc:56: 5:38.24 /home/xry111/sources/12.5/firefox-140.8.0/gfx/skia/skia/modules/skcms/src/Transform_inl.h:160:12: error: use of undeclared identifier '__builtin_ia32_vcvtph2ps256'; did you mean '__builtin_ia32_cvtpd2ps256'? 5:38.24 160 | return __builtin_ia32_vcvtph2ps256((I16)half); 5:38.24 | ^~~~~~~~~~~~~~~~~~~~~~~~~~~ 5:38.24 | __builtin_ia32_cvtpd2ps256 5:38.25 /usr/lib/clang/22/include/avxintrin.h:2187:18: note: '__builtin_ia32_cvtpd2ps256' declared here 5:38.25 2187 | return (__m128)__builtin_ia32_cvtpd2ps256((__v4df) __a); 5:38.25 | ^ 5:38.25 In file included from /home/xry111/sources/12.5/firefox-140.8.0/gfx/skia/skia/modules/skcms/src/skcms_TransformHsw.cc:56: 5:38.26 /home/xry111/sources/12.5/firefox-140.8.0/gfx/skia/skia/modules/skcms/src/Transform_inl.h:160:40: error: cannot initialize a parameter of type '__attribute__((__vector_size__(4 * sizeof(double)))) double' (vector of 4 'double' values) with an rvalue of type 'I16' (vector of 8 'int16_t' values) 5:38.26 160 | return __builtin_ia32_vcvtph2ps256((I16)half); 5:38.26 | ^~~~~~~~~ 5:38.30 2 errors generated.
follow-up: 10 comment:9 by , 7 months ago
comment:10 by , 7 months ago
Replying to Xi Ruoyao:
It seems https://skia-review.googlesource.com/c/skcms/+/1038217.
This has a pretty good shot of breaking Libreoffice as well, since it uses Skia in some places too
comment:11 by , 7 months ago
| Resolution: | → fixed |
|---|---|
| Status: | assigned → closed |
Fixed:
- r13.0-73-g4040af26b0 thunderbird: add a patch to fix FTBFS with LLVM 22
- r13.0-72-gf6dfd82805 thunderbird: use the idiomatic way to patch the vendored rust source
- r13.0-71-g8b137d29d5 thunderbird: simplify the removal of .gitmodules from checksum files and note the upstream issue
- r13.0-70-g99fd6ac90d update to librsvg-2.62.0
- r13.0-69-gf8dcf39bd2 update to cargo-c-0.10.21
- r13.0-68-g0bf7e62cda firefox: add a patch to fix FTBFS with LLVM 22
- r13.0-67-g38ca847519 firefox: use the idiomatic way to patch the vendored rust source
- r13.0-66-ge1fec821f7 update to rustc-1.94.0
- r13.0-65-gdd35278f09 update to spirv-llvm-translator-22.1.0
- r13.0-64-gbdd994e444 update to llvm-22.1.0 and libclc-22.1.0

There seem no longer tarballs for separated projects, only a llvm-project-20.1.0.src.tar.xz monolithic tarball. We need to chdir into the llvm directory and add
-D LLVM_ENABLE_RUNTIMES=compiler-rt -D LLVM_ENABLE_PROJECTS=clangto avoid building many projects not needed by BLFS.The good thing is we no longer need to download 5 tarballs (and host 2 of them on anduin) in this way.