[flang-commits] [flang] Reapply #91116 with fix (PR #93160)

via flang-commits flang-commits at lists.llvm.org
Sun May 26 21:34:34 PDT 2024


https://github.com/NimishMishra approved this pull request.

LGTM. Thanks for this. I'm okay with the approach of having a custom symbol visitor to differentiate between DEF and REF, and then using it to modify `symbolsInNestedRegion`. 

Such a symbol visitor could be a useful tool later on, and maybe we can eventually move it into `Utils.cpp` to share across different components. For now however, I think we can go ahead with the PR.

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


More information about the flang-commits mailing list