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

[Kiran Chandramohan]

kiranchandramohan 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.

The clangir project (https://github.com/llvm/clangir) is being upstreamed. https://discourse.llvm.org/t/rfc-upstreaming-clangir/76587

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