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.

Sign up for GitHub

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

Jump to bottom

llvm-omp-device-info not build when using RUNTIMES instead of PROJECTS #50627

Closed
josemonsalve2 opened this issue Jul 30, 2021 · 3 comments
Closed

llvm-omp-device-info not build when using RUNTIMES instead of PROJECTS #50627

josemonsalve2 opened this issue Jul 30, 2021 · 3 comments
Labels
bugzilla Issues migrated from bugzilla openmp

Comments

@josemonsalve2
Copy link
Contributor

Bugzilla Link 51283
Resolution FIXED
Resolved on Oct 11, 2021 20:29
Version unspecified
OS All
Blocks #51489
Attachments Patch
Fixed by commit(s) 5424cee 60c388a

Extended Description

When building OpenMP with the -DLLVM_ENABLED_RUNTIMES=openmp, instead of -DOPENMP_ENABLED_PROJECTS=openmp, the llvm-omp-device-info tool is not compiled or installed.

This is because the runtimes compilation process removes the -DLLVM_BUILD_TOOLS flag from the second cmake call.
@josemonsalve2
Copy link
Contributor Author

Submitted the review in phabricator.

https://reviews.llvm.org/D107177

@tstellar
Copy link
Collaborator

tstellar commented Aug 3, 2021

Merged: 60c388a

@tstellar
Copy link
Collaborator

mentioned in issue #51489

@tstellar tstellar mentioned this issue Oct 12, 2021
Closed
@llvmbot llvmbot transferred this issue from llvm/llvm-bugzilla-archive Dec 11, 2021
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bugzilla Issues migrated from bugzilla openmp
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@tstellar @josemonsalve2