RFC <link> This patch moves the emission of global variables from DwarfDebug::beginModule() to DwarfDebug::endModule(). Effectively, this changes nothing but the order of debug entities in the resulting DWARF. This is needed to simplify DWARF emission in a context of proper support of function-local static variables (which are globals in terms of LLVM IR) which comes in the next patch (DXXXXXX), making all the globals handled in a single place, together with other global and local entities. For history: this is a part of D114705, see some other motivation points there.

RFC https://discourse.llvm.org/t/rfc-dwarfdebug-fix-and-improve-handling-imported-entities-types-and-static-local-in-subprogram-and-lexical-block-scopes/68544 This patch moves the emission of global variables from `DwarfDebug::beginModule()` to `DwarfDebug::endModule()`. Effectively, this changes nothing but the order of debug entities in the resulting DWARF. This is needed to simplify DWARF emission in a context of proper support of function-local static variables (which are globals in terms of LLVM IR) which comes in the next patch (D144008), making all the globals handled in a single place, together with other global and local entities.

RFC <link>https://discourse.llvm.org/t/rfc-dwarfdebug-fix-and-improve-handling-imported-entities-types-and-static-local-in-subprogram-and-lexical-block-scopes/68544 This patch moves the emission of global variables from `DwarfDebug::beginModule()` to to `DwarfDebug::endModule()`. Effectively, this changes nothing but the order of debug entities in the resulting DWARF. This is needed to simplify DWARF emission in a context of proper support of function-local static variables (which are globals in terms of LLVM IR) which comes in the next patch (DXXXXXXD144008), making all the globals handled in a single place, together with other global and local entities. For history: this is a part of D114705, see some other motivation points there.