[flang-commits] [flang] [flang][OpenMP] Fix construct privatization in default clause (PR #72510)

via flang-commits flang-commits at lists.llvm.org
Wed Mar 27 23:57:46 PDT 2024


NimishMishra wrote:

> > The behaviour of the IR seems fine to me. Is it okay with you? I'll push the revision thereafter.
> 
> @NimishMishra, the test program and resulting IR looks good to me too. Thanks for working on this.

Thanks @luporl . I have pushed the change. I think a similar treatment (to deeply nested constructs) is also needed when we manipulate symbols while resolving directives. However, that problem has already been resolved with your change at https://github.com/llvm/llvm-project/pull/85989, where `shared(z)` uses the parent's copy. Once both the changes go in, the problem should be solved. Is that fine with you? Or should I merge both changes into one patch and then raise a review for the same?

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


More information about the flang-commits mailing list