[Mlir-commits] [flang] [mlir] [flang][OpenACC] Fix implicit data mapping for deviceptr inside host_data (PR #192710)

[Razvan Lupusoru]

razvanlupusoru wrote:

This approach seems reasonable but I am still trying to decide if this part is robust enough:
> In getOriginalDataClauseOpForAlias, for DevicePtrOp clauses additionally check if the clause variable is directly derived from the candidate (e.g., deviceptr operates on embox(candidate)).

In another example I had looked previously the frontend was adding "{acc.declare = #acc.declare<dataClause = acc_deviceptr>" to the variable. I expect this should always happen so if in your case it does not we should see why. This might be a better way to generate deviceptr clause instead of another data clause. Would you consider this approach instead? Let me know if this doesn't pan out for whatever reason.

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