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

[MLIR][OpenMP] Introduce host_eval clause to omp.target #178

Open
wants to merge 1 commit into
base: amd-trunk-dev
Choose a base branch
from

Conversation

skatrak
Copy link

@skatrak skatrak commented Oct 10, 2024

This patch defines a map-like clause named host_eval used to capture host values for use inside of target regions on restricted cases:

  • As num_teams or thread_limit of a nested omp.target operation.
  • As num_threads of a nested omp.parallel operation or as bounds or steps of a nested omp.loop_nest, if it is a target SPMD kernel.

This replaces the following omp.target arguments: trip_count, num_threads, num_teams_lower, num_teams_upper and teams_thread_limit.

Since it's an entry block argument-generating clause, custom region parsers/printers and the BlockArgOpenMPOpInterface are updated.

This patch defines a map-like clause named `host_eval` used to capture host
values for use inside of target regions on restricted cases:
  - As `num_teams` or `thread_limit` of a nested `omp.target` operation.
  - As `num_threads` of a nested `omp.parallel` operation or as bounds or steps
of a nested `omp.loop_nest`, if it is a target SPMD kernel.

This replaces the following `omp.target` arguments: `trip_count`,
`num_threads`, `num_teams_lower`, `num_teams_upper` and `teams_thread_limit`.
@skatrak skatrak force-pushed the users/skatrak/target-passthrough-02-mlir branch from 44b6230 to 719e50c Compare October 11, 2024 13:10
@skatrak skatrak changed the base branch from users/skatrak/target-passthrough-01-block-args to amd-trunk-dev October 11, 2024 13:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant