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

[Flang][OpenMP] Re-implement lowering of DISTRIBUTE PARALLEL DO #135

Merged
merged 2 commits into from
Aug 13, 2024

Commits on Aug 13, 2024

  1. [MLIR][OpenMP] Remove omp.parallel from loop wrapper operations

    This patch removes the `LoopWrapperInterface` from `omp.parallel` and updates
    the semantics of the interface to make loop wrapper restrictions mandatory to
    operations that have it, rather than a role they might optionally take.
    
    MLIR operation verifiers are updated to expect the "hoisted omp.parallel"
    representation for `distribute parallel do`, to be later implemented in place
    of a loop wrapper `omp.parallel`.
    skatrak committed Aug 13, 2024
    Configuration menu
    Copy the full SHA
    3abba41 View commit details
    Browse the repository at this point in the history
  2. [Flang][OpenMP] Re-implement lowering of 'DISTRIBUTE PARALLEL DO'

    This patch updates the Flang lowering process for `distribute parallel do` to
    follow the "hoisted `omp.parallel`" representation. Now temporary allocations
    produced while lowering the loop body reside inside of that operation instead
    of the loop wrappers' parent region.
    
    Special handling of `omp.parallel` with regards to alloca creation is removed,
    as it's no longer necessary to make this distinction. Impacted Lit tests are
    updated according to the new representation.
    skatrak committed Aug 13, 2024
    Configuration menu
    Copy the full SHA
    01c368d View commit details
    Browse the repository at this point in the history