Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[hipSYCL] Add hipSYCL GitHub CI #307

Closed
wants to merge 204 commits into from

Conversation

normallytangent
Copy link

Description

This PR adds a Github CI to test compilation and the CPU target for the BLAS domain with hipSYCL

Replacement for #194

sbalint98 and others added 30 commits July 4, 2022 15:33
…SYCL on Ubuntu 18.04 and 20.04

Squashed commit of the following:

commit 5ee7dd1
Author: Nils Friess <nils.friess@gmail.com>
Date:   Thu Jun 30 11:44:57 2022 +0200

    Also test on Ubuntu 18.04 (requires installing boost from source)

commit 4cfe3aa
Author: Nils Friess <nils.friess@gmail.com>
Date:   Thu Jun 23 22:49:30 2022 +0200

    Renamed workflow file

commit 911b977
Author: Nils Friess <nils.friess@gmail.com>
Date:   Thu Jun 23 22:22:39 2022 +0200

    Run also on pull requests

commit 18670fa
Author: Nils Friess <nils.friess@gmail.com>
Date:   Thu Jun 23 22:08:56 2022 +0200

    Revert "Also test on Ubuntu 22.04 LTS"

    This reverts commit 1912951.
    ROCm currently cannot be installed on 22.04.

commit 1912951
Author: Nils Friess <nils.friess@gmail.com>
Date:   Thu Jun 23 22:02:05 2022 +0200

    Also test on Ubuntu 22.04 LTS

commit 2aca44b
Author: Nils Friess <nils.friess@gmail.com>
Date:   Thu Jun 23 22:01:41 2022 +0200

    Format and remove unused flags etc.

commit c976018
Author: Nils Friess <nils.friess@gmail.com>
Date:   Tue Jun 21 11:13:31 2022 +0200

    Fix typo in CMake command

commit 716a3ff
Author: Nils Friess <nils.friess@gmail.com>
Date:   Tue Jun 21 11:04:53 2022 +0200

    Update apt sources before installing boost

commit 78c0b74
Author: Nils Friess <nils.friess@gmail.com>
Date:   Tue Jun 21 10:51:38 2022 +0200

    Remove commands to install tbb and mkl since they are not necessary

commit c3b1abd
Author: Nils Friess <nils.friess@gmail.com>
Date:   Mon Jun 20 14:25:29 2022 +0200

    Run cmake install as sudo

commit 7f4ddbd
Author: Nils Friess <nils.friess@gmail.com>
Date:   Mon Jun 20 14:15:49 2022 +0200

    Add path to lapack in cmake command

commit 33d23a5
Author: Nils Friess <nils.friess@gmail.com>
Date:   Mon Jun 20 14:12:42 2022 +0200

    Install LAPACK from source

commit f87e040
Author: Nils Friess <nils.friess@gmail.com>
Date:   Mon Jun 20 13:53:46 2022 +0200

    Add command to build oneMKL

commit 159db62
Author: Nils Friess <nils.friess@gmail.com>
Date:   Mon Jun 20 13:53:18 2022 +0200

    Fix typo in cmake flag

commit 4d50f8b
Author: Nils Friess <nils.friess@gmail.com>
Date:   Sun Jun 19 13:58:20 2022 +0200

    Add command to install CBLAS (for simplicity we install libatlas)

commit 3527cd8
Author: Nils Friess <nils.friess@gmail.com>
Date:   Sun Jun 19 13:43:03 2022 +0200

    Add command to install cblas

commit 4a6906c
Author: Nils Friess <nils.friess@gmail.com>
Date:   Sun Jun 19 13:28:53 2022 +0200

    Add path to cmake config of hsa-runtime64 as env variable

commit f6f62a9
Author: Nils Friess <nils.friess@gmail.com>
Date:   Sun Jun 19 13:15:35 2022 +0200

    Add path to amd_comgr cmake config as env variable

commit 6fd04a0
Author: Nils Friess <nils.friess@gmail.com>
Date:   Sun Jun 19 12:52:27 2022 +0200

    Add path to cmake config for AMDDeviceLibs as env variable

commit d9fc104
Author: Nils Friess <nils.friess@gmail.com>
Date:   Sun Jun 19 12:41:25 2022 +0200

    Add path to hip cmake config as environment variable

commit 5125368
Author: Nils Friess <nils.friess@gmail.com>
Date:   Sun Jun 19 12:32:52 2022 +0200

    Install rocblas together with rocm

commit 483bdc6
Author: Nils Friess <nils.friess@gmail.com>
Date:   Sun Jun 19 12:23:05 2022 +0200

    Add rocblas_DIR to cmake command

commit fa58d4b
Author: Nils Friess <nils.friess@gmail.com>
Date:   Sun Jun 19 12:13:32 2022 +0200

    Add rocBLAS CMake config path as environment variable

commit 52065e9
Author: Nils Friess <nils.friess@gmail.com>
Date:   Sun Jun 19 11:00:08 2022 +0200

    Fix hipSYCL build paths

commit bd79ccf
Author: Nils Friess <nils.friess@gmail.com>
Date:   Sun Jun 19 10:45:27 2022 +0200

    Add commands for building oneMKL

commit 4b44a68
Merge: be11ced 6b95323
Author: Nils Friess <nils.friess@gmail.com>
Date:   Wed Jun 8 23:26:20 2022 +0200

    Merge branch 'github-ci-test' of github.com:nilsfriess/oneMKL into github-ci-test

commit be11ced
Author: Nils Friess <nils.friess@gmail.com>
Date:   Wed Jun 8 23:23:11 2022 +0200

    Add hipsycl build (including setup for CUDA and ROCm)

commit 6b95323
Author: Nils Friess <nils.friess@gmail.com>
Date:   Wed Jun 8 23:20:41 2022 +0200

    Remove sudo and add hipsycl build

commit 8378aba
Author: Nils Friess <nils.friess@gmail.com>
Date:   Wed Jun 8 23:03:34 2022 +0200

    Add github workflow test file
rscohn2 and others added 26 commits August 6, 2024 15:42
…get (oneapi-src#511)

* The portFFT backends targets Nvidia by default
* In some situations, this causes a failure whilst compiling (CUDA might not be installed)
* This checks the target is supported before use

This solves the issue in PR oneapi-src#495
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* Rewrites build documentation
  * Splits documentation into several files
* Adds docs for using with CMake.
… for maintainers (oneapi-src#520)

Co-authored-by: Dai-Ni Hsieh <dai-ni.hsieh@intel.com>
Add support for more batch_gemm types to follow the specification.
Some combination using int8 are disabled on some backends due to precision issue.
* INPUT/OUTPUT_STRIDES are deprecated in the oneMKL spec and Intel(R) oneMLK 2024.1 (the latest release)
* Using them causes a warning message to be printed
* This PR uses the new API: FWD/BWD_STRIDES
…eapi-src#503)

* OOP COMPLEX and IP REAL_REAL overload resolution is problematic
  * Inplace real-real overload would be selected when out-of-place complex-complex DFT was intended.
  * With spec update, this PR uses SFINAE to give the expected behaviour for the user.
* Docs incorrectly state that the example builds for Intel, contradicting itself later on.
* This error is corrected.
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
… fix build (oneapi-src#534)

Signed-off-by: JackAKirk <jack.kirk@codeplay.com>
… fix (oneapi-src#525)

Signed-off-by: nscipione <nicolo.scipione@codeplay.com>

* Catch PortBLAS's unsupported exceptions and rethrow as mkl::unimplemented.
* Add missing checks for the device having double support in tests.
… against a DFT backend library (oneapi-src#523)

* Using the DFT backend targets should include the public include directories.
… stage to have device code because of failures on windows (oneapi-src#533)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet