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 HIP builds for the upcoming HIP compiler (SWDEV-310166) #1264

Merged
merged 5 commits into from
Nov 9, 2021

Conversation

atamazov
Copy link
Contributor

@atamazov atamazov commented Nov 7, 2021

This is urgent PR that is expected to unblock hip promotion (https://ontrack-internal.amd.com/browse/SWDEV-310166).

  • [HIP offline build] Remove x86 builtin lib from linking.
  • [COMGR] Remove x86 builtin lib from linking more reliably.
  • By-products
    • [Future ROCm] Disable -Wreserved-identifier for declarations of llvm intrinsics in HIP kernels. Otherwise DEV builds fail at runtime, during HIP compilation (due to -Werror).

@codecov

This comment has been minimized.

src/comgr.cpp Outdated Show resolved Hide resolved
@atamazov
Copy link
Contributor Author

atamazov commented Nov 8, 2021

Reviewers, please go ahead, thanks.

@atamazov atamazov merged commit 1be7b32 into develop Nov 9, 2021
@atamazov atamazov deleted the fix-swdev-310166 branch December 6, 2021 13:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants