Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 822438 - dev-util/intel-graphics-compiler needs porting to LLVM 12+
Summary: dev-util/intel-graphics-compiler needs porting to LLVM 12+
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal
Assignee: Jan Henke
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 822426
  Show dependency tree
 
Reported: 2021-11-08 12:29 UTC by Michał Górny
Modified: 2022-03-27 22:23 UTC (History)
4 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Michał Górny archtester Gentoo Infrastructure gentoo-dev Security 2021-11-08 12:29:01 UTC
12.0.0 has been released back in April.  Please port this package to support it.
Comment 1 Conrad Kostecki gentoo-dev 2021-11-08 13:41:47 UTC
I am seeing in Upstreams Git work for LLVM12 und LLVM13, but I am not able to compile currently with that versions. Anyone an idea?
Comment 2 Conrad Kostecki gentoo-dev 2021-11-08 13:45:45 UTC
According to https://github.com/intel/intel-graphics-compiler/projects/2, LLVM12 should work.
Comment 3 Ionen Wolkens gentoo-dev 2021-11-08 15:22:02 UTC
Not sure what issue you ran into, but this is what I had to do (at least) to get it to build with :13 (haven't tried :12)

- opencl-clang-13.0.0 rough bump
(need -DPREFERRED_LLVM_VERSION=13.0.0 or so)

- spirv-llvm-translator-13.0.0 rough bump
(need -DLLVM_EXTERNAL_SPIRV_HEADERS_SOURCE_DIR="${ESYSROOT}"/usr + spirv-headers dep)

- intel-graphics-compiler-1.0.8744-r1 revbump with workaround, just a quick sed (should patch or find a better workaround):

    sed -i '/EXISTS.*mach-o/c\if(0)' external/llvm/llvm_deps.cmake || die

This would only be needed if it build lld (>=12.0.0), but we're using the system's.

Unsure if it works right (I don't use this nor know much about these packages).

On a side-note, lld belong in RDEPEND too (liblldELF)
$ scanelf -n /usr/lib64/libigc.so.1.0.1
 TYPE   NEEDED FILE 
ET_DYN librt.so.1,liblldELF.so.13,libpthread.so.0,libdl.so.2,libLLVM-13.so,libstdc++.so.6,libm.so.6,libgcc_s.so.1,libc.so.6,ld-linux-x86-64.so.2 /usr/lib64/libigc.so.1.0.1
Comment 4 Ionen Wolkens gentoo-dev 2021-11-08 16:07:24 UTC
(In reply to Ionen Wolkens from comment #3)
> Not sure what issue you ran into, but this is what I had to do (at least) to
> get it to build with :13 (haven't tried :12)
I tried :13 mostly because I didn't want to build :12, but looking at upstream :13 may not be right yet (I see an open 13-only issue).

the mach-o workaround will still be needed with :12 though
Comment 5 Michał Górny archtester Gentoo Infrastructure gentoo-dev Security 2022-03-21 09:29:01 UTC
Ping.
Comment 6 Conrad Kostecki gentoo-dev 2022-03-22 21:38:56 UTC
I haven't forgot about it, but currently short on time to see this through. @Jan: What about you?
Comment 7 Conrad Kostecki gentoo-dev 2022-03-22 23:48:11 UTC
Okay, I think, I have locally a working LLVM-13 newest version. Will submit a PR for review. But will first try to get working newest compute-runtime too.
Comment 8 Larry the Git Cow gentoo-dev 2022-03-27 22:23:13 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=eb7a87df42143a54309e6c16ec23dccaa430cfc9

commit eb7a87df42143a54309e6c16ec23dccaa430cfc9
Author:     Conrad Kostecki <conikost@gentoo.org>
AuthorDate: 2022-03-27 01:17:54 +0000
Commit:     Conrad Kostecki <conikost@gentoo.org>
CommitDate: 2022-03-27 22:21:04 +0000

    dev-util/intel-graphics-compiler: add 1.0.10713
    
    Bug: https://bugs.gentoo.org/817146
    Closes: https://bugs.gentoo.org/822438
    Closes: https://github.com/gentoo/gentoo/pull/24731
    Signed-off-by: Conrad Kostecki <conikost@gentoo.org>

 dev-util/intel-graphics-compiler/Manifest          |  1 +
 ...713-allow-external-spirv-tools-fix-static.patch | 23 ++++++
 ...10713-allow-external-spirv-tools-last-fix.patch | 40 ++++++++++
 .../intel-graphics-compiler-1.0.10713.ebuild       | 87 ++++++++++++++++++++++
 4 files changed, 151 insertions(+)