Skip to content

Fix check for level zero headers - #2366

Merged
ndgrigorian merged 2 commits into
masterfrom
update-getlevelzeroheaders
Aug 19, 2026
Merged

Fix check for level zero headers#2366
ndgrigorian merged 2 commits into
masterfrom
update-getlevelzeroheaders

Conversation

@ndgrigorian

Copy link
Copy Markdown
Collaborator

This PR updates the GetLevelZeroHeaders.cmake

The paths defined previously in parts of the CMake file did not work and were inconsistent, using CMAKE_BINARY_DIR in some places (which worked) and relative paths in others

  • Have you provided a meaningful PR description?
  • Have you added a test, reproducer or referred to an issue with a reproducer?
  • Have you tested your changes locally for CPU and GPU devices?
  • Have you made sure that new changes do not introduce compiler warnings?
  • Have you checked performance impact of proposed changes?
  • Have you added documentation for your changes, if necessary?
  • Have you added your changes to the changelog?
  • If this PR is a work in progress, are you opening the PR as a draft?

@ndgrigorian
ndgrigorian force-pushed the update-getlevelzeroheaders branch from 28d45cb to 0013c08 Compare August 19, 2026 01:59
@github-actions

github-actions Bot commented Aug 19, 2026

Copy link
Copy Markdown

Deleted rendered PR docs from intelpython.github.com/dpctl, latest should be updated shortly. 🤞

@coveralls

coveralls commented Aug 19, 2026

Copy link
Copy Markdown
Collaborator

Coverage Status

coverage: 74.462%. remained the same — update-getlevelzeroheaders into master

@antonwolfy antonwolfy left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ndgrigorian
ndgrigorian merged commit 6b3c588 into master Aug 19, 2026
101 checks passed
@ndgrigorian
ndgrigorian deleted the update-getlevelzeroheaders branch August 19, 2026 08:49
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.

3 participants