[flang-commits] [flang] [flang][OpenMP] Store DECLARE_TARGET information in WithOmpDeclarative (PR #201103)

[Sergio Afonso via flang-commits]

https://github.com/skatrak commented:

Thanks Krzysztof. Only minor comments from me, otherwise I think this looks good.

Also, I think the PR description should be updated because there's a problem in the sentence "When a symbol apperars in DECLARE_TARGET (**explicitly or explicitly**),...". I think this is only for explicit DECLARE_TARGET. At the moment, we're doing the implicit DECLARE_TARGET marking of functions based on usage in target regions after lowering to MLIR (in the `MarkDeclareTargetPass`).

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