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

[OpenMP][Fix] Fix test initialization #74801

Merged
merged 1 commit into from
Dec 8, 2023
Merged

[OpenMP][Fix] Fix test initialization #74801

merged 1 commit into from
Dec 8, 2023

Conversation

doru1004
Copy link
Contributor

@doru1004 doru1004 commented Dec 8, 2023

Fix test initialization

@llvmbot llvmbot added the openmp:libomptarget OpenMP offload runtime label Dec 8, 2023
@doru1004 doru1004 requested a review from jhuber6 December 8, 2023 03:17
jhuber6
jhuber6 approved these changes Dec 8, 2023
View reviewed changes
Copy link
Contributor

@jhuber6 jhuber6 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry, didn't notice the missing comma.

@doru1004 doru1004 merged commit 5fc76e6 into llvm:main Dec 8, 2023
5 checks passed
@doru1004 doru1004 deleted the fix-test-initializations-3 branch December 8, 2023 03:20
@doru1004
Copy link
Contributor Author

doru1004 commented Dec 8, 2023

Sorry, didn't notice the missing comma.

neither did I :)

@jdoerfert
Copy link
Member

Why was this necessary?

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

@jhuber6 jhuber6 jhuber6 approved these changes

Assignees
No one assigned
Labels
openmp:libomptarget OpenMP offload runtime
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@doru1004 @jdoerfert @jhuber6 @llvmbot