[Mlir-commits] [mlir] [mlir][flang] Improve handling of fortran module variables. (PR #91604)

[Abid Qadeer]

abidh wrote:

> Thanks @abidh for separating out this patch.
> 
> > Take the example of namespace in c++ below. The debug metadata was generated by clang. You can see that scope of the global variable a is set to namespace (and not to compile unit) but it still gets added to globals in DICompileUnit.
> 
> Is it possible to have common handling for Namespaces and Modules or will Namespace be another `if` case?

The namespaces are a bit different as they can be fragmented in multiple files. So their handling may indeed be slightly different if ever a C++ frontend targets mlir.

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