[all-commits] [llvm/llvm-project] f1d773: [Flang][OpenMP] Remove use of non reference values...

[Akash Banerjee via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: f1d773863dc2707d96ce0f506801b363f8b079cf
      https://github.com/llvm/llvm-project/commit/f1d773863dc2707d96ce0f506801b363f8b079cf
  Author: Akash Banerjee <akash.banerjee at amd.com>
  Date:   2023-11-24 (Fri, 24 Nov 2023)

  Changed paths:
    M flang/lib/Lower/OpenMP.cpp
    M flang/test/Lower/OpenMP/FIR/array-bounds.f90
    M flang/test/Lower/OpenMP/FIR/target.f90
    M flang/test/Lower/OpenMP/array-bounds.f90
    M flang/test/Lower/OpenMP/target.f90
    M mlir/include/mlir/Dialect/OpenMP/OpenMPOps.td
    M mlir/lib/Dialect/OpenMP/IR/OpenMPDialect.cpp
    M mlir/lib/Target/LLVMIR/Dialect/OpenMP/OpenMPToLLVMIRTranslation.cpp
    M mlir/test/Dialect/OpenMP/invalid.mlir
    R openmp/libomptarget/test/offloading/fortran/basic-target-region-array.f90
    M openmp/libomptarget/test/offloading/fortran/constant-arr-index.f90
    A openmp/libomptarget/test/offloading/fortran/target-region-implicit-array.f90

  Log Message:
  -----------
  [Flang][OpenMP] Remove use of non reference values from MapInfoOp (#72444)

This patch removes the val field from the `MapInfoOp`.

Previously when lowering `TargetOp`, the bounds information for the
`BoxValues` were also being mapped. Instead these ops are now cloned
inside the target region to prevent mapping of non reference typed
values.