[llvm-branch-commits] [flang] [llvm] [mlir] [MLIR][OpenMP] Add OpenMPToLLVMIRTranslation support for is_device_ptr (PR #169367)

[Kareem Ergawy via llvm-branch-commits]

================
@@ -1146,7 +1146,8 @@ bool ClauseProcessor::processIsDevicePtr(
           const parser::CharBlock &source) {
         mlir::Location location = converter.genLocation(source);
         mlir::omp::ClauseMapFlags mapTypeBits =
-            mlir::omp::ClauseMapFlags::storage;
+            mlir::omp::ClauseMapFlags::return_param |
----------------
ergawy wrote:

Why is the `return_param` flag needed here? I think we can only use `is_device_ptr` here and leave the mapping to `llvm::omp::OpenMPOffloadMappingFlags::OMP_MAP_RETURN_PARAM`  be handled by the MLIR to LLVM translation as you do below in `convertClauseMapFlags`.

https://github.com/llvm/llvm-project/pull/169367