Skip to content

Fix fetching L0 and CUDA headers and add UMF_CUDA_INCLUDE_DIR #832

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

Conversation

ldorau
Copy link
Contributor

@ldorau ldorau commented Oct 24, 2024

Description

Fix fetching L0 and CUDA headers and add UMF_CUDA_INCLUDE_DIR.

Fetch L0 loader only if needed i.e.:
if building L0 provider is ON and L0 headers are not provided by the user (via setting UMF_LEVEL_ZERO_INCLUDE_DIR).

Fetch CUDA only if needed i.e.:
if building CUDA provider is ON and CUDA headers are not provided by the user (via setting UMF_CUDA_INCLUDE_DIR).

Ref: #825

Fixes:: #827

Checklist

  • Code compiles without errors locally
  • All tests pass locally
  • CI workflows execute properly

@ldorau ldorau requested a review from a team as a code owner October 24, 2024 07:03
@ldorau ldorau force-pushed the Fix_fetching_L0_and_CUDA_headers_and_add_UMF_CUDA_INCLUDE_DIR branch from a88268c to 10adcdb Compare October 24, 2024 07:13
@ldorau ldorau force-pushed the Fix_fetching_L0_and_CUDA_headers_and_add_UMF_CUDA_INCLUDE_DIR branch from 10adcdb to a11f927 Compare October 25, 2024 06:45
@ldorau ldorau force-pushed the Fix_fetching_L0_and_CUDA_headers_and_add_UMF_CUDA_INCLUDE_DIR branch 2 times, most recently from 151d58e to b887d38 Compare October 25, 2024 08:29
@ldorau ldorau marked this pull request as draft October 25, 2024 08:30
@ldorau ldorau force-pushed the Fix_fetching_L0_and_CUDA_headers_and_add_UMF_CUDA_INCLUDE_DIR branch 5 times, most recently from 8b9bb5e to f2615bc Compare October 25, 2024 10:06
@ldorau ldorau marked this pull request as ready for review October 25, 2024 10:07
@ldorau
Copy link
Contributor Author

ldorau commented Oct 25, 2024

@lukaszstolarczuk resolve your issues, please

Fetch L0 loader only if needed i.e.:
if building L0 provider is ON and L0 headers are not provided
by the user (via setting UMF_LEVEL_ZERO_INCLUDE_DIR).

Fetch CUDA only if needed i.e.:
if building CUDA provider is ON and CUDA headers are not provided
by the user (via setting UMF_CUDA_INCLUDE_DIR).

Ref: oneapi-src#825

Fixes: oneapi-src#827

Signed-off-by: Lukasz Dorau <[email protected]>
@ldorau ldorau force-pushed the Fix_fetching_L0_and_CUDA_headers_and_add_UMF_CUDA_INCLUDE_DIR branch from f2615bc to 89025a1 Compare October 25, 2024 13:40
@ldorau
Copy link
Contributor Author

ldorau commented Oct 25, 2024

Rebased

@ldorau ldorau merged commit 43e9af0 into oneapi-src:main Oct 28, 2024
74 checks passed
@ldorau ldorau deleted the Fix_fetching_L0_and_CUDA_headers_and_add_UMF_CUDA_INCLUDE_DIR branch October 28, 2024 09:40
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.

5 participants