Skip to content

Conversation

ashermancinelli
Copy link
Contributor

This test utility was showing up in -Wunused builds and I cannot find any references to it. Remove it.

This test utility was showing up in -Wunused builds and I cannot find
any references to it. Remove it.
@ashermancinelli ashermancinelli self-assigned this Oct 1, 2025
@llvmbot llvmbot added openmp:libomp OpenMP host runtime openmp:libomptarget OpenMP offload runtime labels Oct 1, 2025
@jhuber6 jhuber6 requested a review from jplehr October 1, 2025 15:02
@ashermancinelli
Copy link
Contributor Author

Feel free to close this or we can use [[maybe_unused]], this is just to keep the Wunused build clean and I'm not an omp dev 😄

@jplehr
Copy link
Contributor

jplehr commented Oct 1, 2025

Thank you for opening the PR. I believe we missed implementing the listener for it.
Let me pull in @mhalk to get his thoughts.

@jplehr
Copy link
Contributor

jplehr commented Oct 2, 2025

I put up #161650 that enabled the handler.

I don't know why it was commented out TBH.

@ashermancinelli
Copy link
Contributor Author

Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

openmp:libomp OpenMP host runtime openmp:libomptarget OpenMP offload runtime

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants