Skip to content

[L0] Use relative includes for adapter internals#1548

Merged
kbenzie merged 1 commit intooneapi-src:mainfrom
kbenzie:benie/l0-fix-decoupled-use-case
Apr 25, 2024
Merged

[L0] Use relative includes for adapter internals#1548
kbenzie merged 1 commit intooneapi-src:mainfrom
kbenzie:benie/l0-fix-decoupled-use-case

Conversation

@kbenzie
Copy link
Contributor

@kbenzie kbenzie commented Apr 25, 2024

Fix build issues when using decoupled adapter fetch in intel/llvm by using relative include paths instead of longer explicit paths. This was causing redefinitions of objects. This patch also adds a missing #pragma once from the adapter.hpp header.

@kbenzie kbenzie requested a review from a team as a code owner April 25, 2024 11:09
@kbenzie
Copy link
Contributor Author

kbenzie commented Apr 25, 2024

This should resolve build issues like this.

@kbenzie kbenzie added the v0.9.x Include in the v0.9.x release label Apr 25, 2024
@github-actions github-actions bot added the level-zero L0 adapter specific issues label Apr 25, 2024
Fix build issues when using decoupled adapter fetch in intel/llvm by
using relative include paths instead of longer explicit paths. This was
causing redefinitions of objects. This patch also adds a missing
`#pragma once` from the `adapter.hpp` header.
@kbenzie kbenzie force-pushed the benie/l0-fix-decoupled-use-case branch from 1ee9f76 to 275c753 Compare April 25, 2024 12:07
@kbenzie kbenzie merged commit 196f62f into oneapi-src:main Apr 25, 2024
kbenzie added a commit to kbenzie/unified-runtime that referenced this pull request May 3, 2024
…d-use-case

[L0] Use relative includes for adapter internals
@kbenzie kbenzie deleted the benie/l0-fix-decoupled-use-case branch June 11, 2024 15:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

level-zero L0 adapter specific issues v0.9.x Include in the v0.9.x release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants