[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