[all-commits] [llvm/llvm-project] 41c41d: [flang][OpenMP] Move `createMapInfoOp` to `OpenMP-...

[Kareem Ergawy via All-commits]

  Branch: refs/heads/users/ergawy/move_createmapinfoop
  Home:   https://github.com/llvm/llvm-project
  Commit: 41c41dc3fa88e36be4ab555230d6fd492f1e7c85
      https://github.com/llvm/llvm-project/commit/41c41dc3fa88e36be4ab555230d6fd492f1e7c85
  Author: ergawy <kareem.ergawy at amd.com>
  Date:   2025-08-26 (Tue, 26 Aug 2025)

  Changed paths:
    M flang/include/flang/Support/OpenMP-utils.h
    M flang/lib/Lower/OpenMP/ClauseProcessor.cpp
    M flang/lib/Lower/OpenMP/Utils.cpp
    M flang/lib/Lower/OpenMP/Utils.h
    M flang/lib/Support/CMakeLists.txt
    M flang/lib/Support/OpenMP-utils.cpp

  Log Message:
  -----------
  [flang][OpenMP] Move `createMapInfoOp` to `OpenMP-utils.h`

This moves `createMapInfoOp` from `flang/lib/Lower/OpenMP/Utils.h` to `flang/include/flang/Support/OpenMP-utils.h`.

Context: I am working on upstreaming (from AMD's downstream fork) support for `do concurrent` mapping to the GPU. This means that `DoConcurrentConversion.cpp` needs access to `createMapInfoOp`. Hence, we moved it downstream to a shared location between that is linked by both `FlangOpenMPTransforms` (where the `do concurrent` pass lives) and `FortranLower` (where `createMapInfoOp` originally were).

The issue now is that we have to link in both the FIR and OpenMP MLIR dialects which is not ideal to link with a suport library like `FortranSupport`.

Note that, so far, upstream `DoConcurrentConversion.cpp` does not reference `createMapInfoOp` yet. Follow-up PRs will upstream this later.


  Commit: 04ef4d618643eb1e54048fa16e9d41efac235fc2
      https://github.com/llvm/llvm-project/commit/04ef4d618643eb1e54048fa16e9d41efac235fc2
  Author: ergawy <kareem.ergawy at amd.com>
  Date:   2025-08-26 (Tue, 26 Aug 2025)

  Changed paths:
    M flang/include/flang/Support/OpenMP-utils.h

  Log Message:
  -----------
  Add docs


  Commit: ed16ebee7eb92957eedc027253dee976480a35aa
      https://github.com/llvm/llvm-project/commit/ed16ebee7eb92957eedc027253dee976480a35aa
  Author: ergawy <kareem.ergawy at amd.com>
  Date:   2025-08-26 (Tue, 26 Aug 2025)

  Changed paths:
    M flang/include/flang/Support/OpenMP-utils.h
    A flang/include/flang/Utils/OpenMP.h
    M flang/lib/CMakeLists.txt
    M flang/lib/Lower/CMakeLists.txt
    M flang/lib/Lower/OpenMP/ClauseProcessor.cpp
    M flang/lib/Lower/OpenMP/OpenMP.cpp
    M flang/lib/Lower/OpenMP/Utils.cpp
    M flang/lib/Support/CMakeLists.txt
    M flang/lib/Support/OpenMP-utils.cpp
    A flang/lib/Utils/CMakeLists.txt
    A flang/lib/Utils/OpenMP.cpp

  Log Message:
  -----------
  Add `FortranUtils` libarary.


Compare: https://github.com/llvm/llvm-project/compare/518312a4bbe6...ed16ebee7eb9

To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications