[flang-commits] [flang] [flang][OpenMP] Lower `target .. private(..)` to `omp.private` ops (PR #94195)

[Kiran Chandramohan via flang-commits]

kiranchandramohan wrote:

> > What I am worried about is whether we want a different style entry in the PrivateClause Operation so that the firstprivate variable can be laid out properly in the privatisation part of the `task_t` structure.
> 
> I am afraid I dont really understands what you mean here, specifically when you say "different style entry in PrivateClause Operation". Could you please elaborate?

To layout a copy of the original variable in the task datastructure, I am assuming you will need some kind of size, offset and alignment information. With a PrivateClause operation we have a sequence of MLIR operations. It is not clear whether we can deduce the necessary information for laying out a firstprivate variable in the task datastructure. 

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