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

Fix compilation errors with musl libc #463

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

AngryLoki
Copy link

  1. error.h is a GNU extension and is not provided by musl. Replaced with fprintf.
  2. NULL is defined as a value of type std::nullptr_t, reinterpret_cast from std::nullptr_t to void ** is not allowed[1]. But static_cast is allowed.

[1] reinterpret_cast conversion - cppreference.com

1. `error.h` is a GNU extension and is not provided by musl. Replaced with fprintf.
2. NULL is defined to `std::nullptr_t`, reinterpret_cast from `std::nullptr_t` to `void **` is not allowed. But static_cast is allowed.[1]

[1] https://en.cppreference.com/w/cpp/language/reinterpret_cast#:~:text=Note%20that%20the%20null%20pointer,be%20used%20for%20this%20purpose.

Signed-off-by: Sv. Lockal <[email protected]>
AngryLoki added a commit to AngryLoki/gentoo that referenced this pull request May 22, 2024
Upstream pull-requests:
* ROCm/clr#83
* ROCm/hip-tests#463

Signed-off-by: Sv. Lockal <[email protected]>
littlewu2508 pushed a commit to littlewu2508/gentoo that referenced this pull request May 23, 2024
Upstream pull-requests:
* ROCm/clr#83
* ROCm/hip-tests#463

Signed-off-by: Sv. Lockal <[email protected]>
@AngryLoki AngryLoki closed this May 27, 2024
@AngryLoki
Copy link
Author

AngryLoki commented May 27, 2024

(accidentally closed wrong pull-request)

@AngryLoki AngryLoki reopened this May 27, 2024
AngryLoki added a commit to AngryLoki/gentoo that referenced this pull request Jun 11, 2024
Upstream pull-requests:
* ROCm/clr#83
* ROCm/hip-tests#463

Signed-off-by: Sv. Lockal <[email protected]>
negril pushed a commit to negril/gentoo that referenced this pull request Jun 25, 2024
Upstream pull-requests:
* ROCm/clr#83
* ROCm/hip-tests#463

Signed-off-by: Sv. Lockal <[email protected]>
Signed-off-by: Paul Zander <[email protected]>
negril pushed a commit to negril/gentoo that referenced this pull request Jun 25, 2024
Upstream pull-requests:
* ROCm/clr#83
* ROCm/hip-tests#463

Signed-off-by: Sv. Lockal <[email protected]>
Signed-off-by: Paul Zander <[email protected]>
negril pushed a commit to negril/gentoo that referenced this pull request Jun 25, 2024
Upstream pull-requests:
* ROCm/clr#83
* ROCm/hip-tests#463

Signed-off-by: Sv. Lockal <[email protected]>
gentoo-bot pushed a commit to gentoo/gentoo that referenced this pull request Jun 26, 2024
Upstream pull-requests:
* ROCm/clr#83
* ROCm/hip-tests#463

Signed-off-by: Sv. Lockal <[email protected]>
Signed-off-by: Sam James <[email protected]>
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.

1 participant