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

[via flang-commits]

vdonaldson wrote:

> I can move it into instantiateVar. Would that be better?

Probably yes, if the propagation code is fairly simple.

But I'm still inclined to think it would be better though to do this earlier if that is reasonable, which it might or might not be. One idea there is to explicitly save the incoming `scope` in the two `SymbolDependenceAnalysis` constructors and then in `analyze` have an early exit if the scope of the symbol being analyzed does not match that scope. I'm not completely sure if that would be ok though.

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