[flang-commits] [flang] [Flang][OpenMP] Fix implicit symbol resolution for USE-renamed arrays (PR #189215)

[Kiran Chandramohan via flang-commits]

https://github.com/kiranchandramohan commented:

There are some unrelated formatting changes in the patch. Could you remove them?

Could you expand the summary of the patch to explain the changes?

This change will make the symbol in the OpenMP region refer to the correct symbol in the module. What I was worried about was whether this will cause a problem if the array is modified (via an allocation) just before the OpenMP region, will pointing to the symbol (via host-association) in the module cause it get the array with the previous allocation?

I think, Ideally (in this test), the symbol in the OpenMP region should refer to the renamed symbol in `subroutine sub`.

If you use `-fdebug-dump-symbols` you can see the host-associations. May be this is the write test for this bug as well.

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