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

[OpenMP] Add num_threads clause list format and strict modifier support #85466

Open
wants to merge 9 commits into
base: main
Choose a base branch
from

Conversation

TerryLWilmarth
Copy link
Contributor

Add support to the runtime for 6.0 spec features that allow num_threads clause to take a list, and also make use of the strict modifier. Provides new compiler interface functions for these features.

Add support to the runtime for 6.0 spec features that allow
num_threads clause to take a list, and also make use of the strict
modifier.  Provides new compiler interface functions for these
features.
@llvmbot llvmbot added the openmp:libomp OpenMP host runtime label Mar 15, 2024
@shiltian
Copy link
Contributor

Can we have tests?

@shiltian shiltian changed the title Add num_threads clause list format and strict modifier support [OpenMP] Add num_threads clause list format and strict modifier support Mar 27, 2024
Copy link

github-actions bot commented Mar 30, 2024

✅ With the latest revision this PR passed the C/C++ code formatter.

Copy link
Contributor

@shiltian shiltian left a comment

Choose a reason for hiding this comment

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

LGTM. Will the front end support be upstreamed as well?

@@ -1268,6 +1268,11 @@ kmp_set_disp_num_buffers 890
__kmpc_atomic_val_8_cas_cpt 2158
%endif

# No longer need to put ordinal numbers
Copy link
Contributor

Choose a reason for hiding this comment

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

This will not cause issue on Windows anymore?

Copy link
Contributor

Choose a reason for hiding this comment

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

Right, you can add exported symbols just by name now. Older symbols keep their ordinal number.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
openmp:libomp OpenMP host runtime
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants