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

[Issue]: Location of half.hpp #292

Closed
trixirt opened this issue Jan 21, 2024 · 4 comments
Closed

[Issue]: Location of half.hpp #292

trixirt opened this issue Jan 21, 2024 · 4 comments
Assignees
Labels
bug Something isn't working

Comments

@trixirt
Copy link

trixirt commented Jan 21, 2024

Problem Description

Same issue as MIOpen

I am packaging rpp for Fedora, the addition of the half/ makes it impossible to set the location of half.hpp in configury.

Operating System

Fedora Rawhide

CPU

x86_64

GPU

AMD Instinct MI210

Other

No response

ROCm Version

ROCm 6.0.0

ROCm Component

rpp

Steps to Reproduce

No response

(Optional for Linux users) Output of /opt/rocm/bin/rocminfo --support

No response

Additional Information

No response

@kiritigowda kiritigowda self-assigned this Jan 24, 2024
@kiritigowda kiritigowda added the bug Something isn't working label Jan 24, 2024
@kiritigowda
Copy link
Collaborator

kiritigowda commented Jan 24, 2024

@trixirt we have FindHalf to locate half headers. If you add additional location here -

${ROCM_PATH}/include
should find the library on your setup.

If you provide Half default on Fedora, I can add it or you can create a PR to make it available for all.

@kiritigowda
Copy link
Collaborator

@trixirt were you able to resolve this issue?

@AngryLoki
Copy link

@trixirt , I think distro maintainers should install half.hpp into /usr/include/half/. I encountered this issue in Gentoo, and I will update ebuilds there in that way.

Reasoning: https://github.com/ROCm/half is a project under ROCm umbrella. MIOpen, MIVisionX, AMDMIGraphX, rpp, MIFin, rocAL and maybe other ROCm projects just decided to include it as <half/half.hpp>. I see this header in other projects, but again, mostly as <half/half.hpp>.

@kiritigowda
Copy link
Collaborator

@trixirt --- With the cmake you can also use the -D HALF_DIRor ENV Variable HALF_DIR to set path to include. If you still face this issue reopen.

-D HALF_DIR /PATH/TO/half/half.hpp

${HALF_DIR}/include

$ENV{HALF_DIR}/include

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants