FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Use ONEAPI_ROOT env variable also for looking up MKL Installation by 9prady9 · Pull Request #3150 · arrayfire/arrayfire · GitHub

Repository navigation

Use ONEAPI_ROOT env variable also for looking up MKL Installation - #3150

Merged
9prady9 merged 1 commit into
arrayfire:masterfrom
9prady9:oneapi_root
Jun 22, 2021
Merged

9prady9 merged 1 commit into
arrayfire:masterfrom
9prady9:oneapi_root

Conversation

9prady9 commented Jun 22, 2021

Copy link
Copy Markdown
Member

Description

If the user sets-up MKL using Intel provided setup scripts prior to building ArrayFire, all environment variables are taken care of by the Intel setup scripts. This change addresses the scenario where oneAPI basetoolkit is installed but the setup script is not run before building ArrayFire.

With this additional environment variable based lookup, in the case MKL setup script is not run, ArrayFire CMake tries to locate MKL based on ONEAPI installation root. This change essentially enables easier look up of MKL installation for building ArrayFire using oneMKL. However, note that to run the built tests/examples the user needs to make sure oneMKL libraries are available via PATH or LD_LIBRARY_PATH for runtime programs to find them.

Changes to Users

None to library users. An minor ease of use enabler for ArrayFire developers.

Checklist

  • Rebased on latest master
  • Code compiles
  • [ ] Tests pass
  • [ ] Functions added to unified API
  • [ ] Functions documented

9prady9 added this to the 3.8.1 milestone Jun 22, 2021
9prady9 requested a review from umar456 June 22, 2021 07:12
9prady9 merged commit a7c6950 into arrayfire:master Jun 22, 2021
9prady9 deleted the oneapi_root branch June 22, 2021 15:11
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants


Back | FazBrowse Home | New Git URL