[flang-commits] [flang] [flang][hlfir] Do not emit extra declare for dummy used in BLOCK (PR #69184)

[via flang-commits]

vdonaldson wrote:

A simple update to `SymbolDependenceAnalysis` to compare `scope` s works with the vast majority of tests because module and host syms have local syms that provide indirect access to their ultimate syms.

However, the `scope` comparison causes lit tests `explicit-interface-results` and `module-single-point-of-def` to fail due to atypical module variable accesses -  for a function result in the former case; and for a namelist reference in the latter case. 

So I'm ok with either the original change, or your possible alternate fix that moves the check near the beginning of `instantiateVar` if that one turns out to be better.

Thanks for the fix.

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