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

[libomptarget][flang] Explicitly pass the OpenMP device libraries to tests #76796

Merged
merged 1 commit into from
Jan 4, 2024

Conversation

DominikAdamski
Copy link
Contributor

This pull request is a follow-up of patch:
#68225 and it explicitly specifies OpenMP device libraries for Fortran OpenMP tests.

…tests

This pull request is a follow-up of patch:
llvm#68225 and it explicitly
specifies OpenMP device libraries for Fortran OpenMP tests.
@ergawy
Copy link
Member

ergawy commented Jan 3, 2024

LGTM.

However, this change will not be effective until we support nogpulib flag in flang, right?

@DominikAdamski
Copy link
Contributor Author

@ergawy You are right. We need support for -nogpulib as well. This patch adds only libomptarget.devicertl.a to Fortran OpenMP test compilation commands.

@DominikAdamski DominikAdamski merged commit 0cdaadf into llvm:main Jan 4, 2024
4 checks passed
@DominikAdamski DominikAdamski deleted the flang_fix_openmp_test branch January 4, 2024 07:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
openmp:libomptarget OpenMP offload runtime
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants