Skip to content

Conversation

@DominikAdamski
Copy link
Contributor

Verify how clang handles dist_schedule(static, block_chunk) and schedule(static, thread_chunk) clauses for OpenMP offload loop workshare pragmas.

Verify how clang handles `dist_schedule(static, block_chunk)`
and `schedule(static, thread_chunk)` clauses for OpenMP offload
loop workshare pragmas.
Copy link
Member

@jdoerfert jdoerfert left a comment

Choose a reason for hiding this comment

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

LG

@jdoerfert jdoerfert merged commit b9a41b9 into llvm:main Mar 19, 2024
uweigand added a commit that referenced this pull request Mar 20, 2024
The test added by #83261
has been consistently failing.  Mark as UNSUPPORTED just like on
x86_64 and aarch64.
@uweigand
Copy link
Member

This test has been consistently failing on s390x-ibm-linux-gnu and s390x-ibm-linux-gnu-LTO. I've now marked those two targets as unsupported, just like x86_64 and aarch64 already were.

I notice that a significant number of the offloading tests are all marked as UNSUPPORTED on the "server" CPU targets (x86_64, aarch64 and s390x) - maybe this could be abstracted in some better way?

chencha3 pushed a commit to chencha3/llvm-project that referenced this pull request Mar 23, 2024
…3261)

Verify how clang handles `dist_schedule(static, block_chunk)` and
`schedule(static, thread_chunk)` clauses for OpenMP offload loop
workshare pragmas.
chencha3 pushed a commit to chencha3/llvm-project that referenced this pull request Mar 23, 2024
The test added by llvm#83261
has been consistently failing.  Mark as UNSUPPORTED just like on
x86_64 and aarch64.
@jdoerfert
Copy link
Member

I notice that a significant number of the offloading tests are all marked as UNSUPPORTED on the "server" CPU targets (x86_64, aarch64 and s390x) - maybe this could be abstracted in some better way?

Yes, we should have "feature flags" rather than listing targets. Here it would be "required teams".

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