[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[bug#55948] [PATCH 0/2] gnu: Update rocm to 5.1.3.
From: |
John Kehayias |
Subject: |
[bug#55948] [PATCH 0/2] gnu: Update rocm to 5.1.3. |
Date: |
Thu, 16 Jun 2022 15:43:38 +0000 |
Hi Ludo’ and Lars,
Thanks for the review and assistance (Lars and I discussed this on IRC a few
days ago).
------- Original Message -------
On Thursday, June 16th, 2022 at 8:17 AM, Ludovic Courtès <ludo@gnu.org> wrote:
> > diff --git a/gnu/packages/llvm.scm b/gnu/packages/llvm.scm
> > index 7b197bb67b..f5d3eac588 100644
> > --- a/gnu/packages/llvm.scm
> > +++ b/gnu/packages/llvm.scm
> > @@ -1127,14 +1127,9 @@ (define-public llvm-for-rocm
> > (search-patches "llvm-roc-5.0.0-linkdl.patch"
> > "llvm-roc-4.0.0-remove-isystem-usr-include.patch"))))
> > (arguments
> > - (substitute-keyword-arguments (package-arguments llvm-12)
> > - ((#:phases phases '%standard-phases)
> > - `(modify-phases ,phases
> > - (add-after 'unpack 'chdir
> > - (lambda _
> > - (chdir "llvm")))))
> > + (substitute-keyword-arguments (package-arguments llvm-14)
> > ((#:configure-flags flags)
> > - ''("-DLLVM_ENABLE_PROJECTS=llvm;clang;lld"
> > + #~(list"-DLLVM_ENABLE_PROJECTS=llvm;clang;lld"
> > "-DLLVM_TARGETS_TO_BUILD=AMDGPU;X86"
> > "-DCMAKE_SKIP_BUILD_RPATH=FALSE"
> > "-DCMAKE_BUILD_WITH_INSTALL_RPATH=FALSE"
>
>
> John, WDYT?
>
Fine with me! (I'm assuming Lars had built this version locally as this just
seems like a clean up and better inheritance structure.)
> > -(define-public rocclr-4
> > - (package
> > - (name "rocclr-4")
> > - (version "4.3.0")
>
> Fine with me; John?
>
Likewise! For some context, Lars and I had discussed on IRC about ROCm dropping
support in later v4 (and now v5) for hardware he uses. Unfortunately, figuring
out what is supported or works in this space is very difficult as they cater to
more workstation setups. We have time machine and guix past at least, though
maybe we can keep older versions explicitly. Since Lars might be one of the few
(only?) that uses the older ROCm, I left it to him. rocclr v4 is probably not
useful on its own, so I'm fine with dropping it.
> Lars, feel free to push these if John agrees.
>
> Ludo’.
By the way, I forgot I had to make a kernel change to use this, see
https://issues.guix.gnu.org/55111 Let me try again with these patches having
landed and report back on that issue directly at #55111.
Thanks everyone!
John