================
@@ -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
_______________________________________________
llvm-branch-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits

Reply via email to