[all-commits] [llvm/llvm-project] 8a9e76: [Flang][MLIR] - Handle the mapping of subroutine a...

[Pranav Bhandarkar via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 8a9e767fa690e0232db2cfa8576f53b48ae53f30
      https://github.com/llvm/llvm-project/commit/8a9e767fa690e0232db2cfa8576f53b48ae53f30
  Author: Pranav Bhandarkar <pranav.bhandarkar at amd.com>
  Date:   2025-05-12 (Mon, 12 May 2025)

  Changed paths:
    M flang/include/flang/Optimizer/Builder/DirectivesCommon.h
    M flang/lib/Lower/OpenMP/OpenMP.cpp
    M flang/lib/Optimizer/OpenMP/MapInfoFinalization.cpp
    A flang/test/Lower/OpenMP/map-character.f90

  Log Message:
  -----------
  [Flang][MLIR] - Handle the mapping of subroutine arguments when they are subsequently used inside the region of  an `omp.target` Op (#134967)

This is a fix for https://github.com/llvm/llvm-project/issues/134912
which is a problem with mapping `fir.boxchar<k>` type values to the
target i.e an `omp.target` op.

There really are two problems. Fixing the first exposed the second. The
first problem is that OpenMP lowering of maps in `omp.target` in Flang
cannot handle the mapping of a value that doesnt have a defining
operation. In other words, a value that is a block argument. This is handled
by mapping the value using a `MapInfoOp`.
The second problem this fixes is that it adds bounds to `omp.map.info`
ops that map `fir.char<k, ?>` types by extracting the length from the
corresponding `fir.boxchar`



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