Addressed review comments.

1. Skip variables if module list is empty.

2. Add a missing line at the end of a file.

Author: abidh

flang/lib/Optimizer/Transforms/AddDebugInfo.cpp

@@ -155,9 +155,11 @@ void AddDebugInfoPass::handleGlobalOp(fir::GlobalOp globalOp,
   // declared. We are using a best guess of line - 1 where line is the source
   // line of the first member of the module that we encounter.
 
-  if (!result.second.modules.empty())
-    scope = getOrCreateModuleAttr(result.second.modules[0], fileAttr, scope,
-                                  line - 1, !globalOp.hasInitializationBody());
+  if (result.second.modules.empty())
+    return;
+
+  scope = getOrCreateModuleAttr(result.second.modules[0], fileAttr, scope,
+                                line - 1, !globalOp.hasInitializationBody());
 
   auto diType = typeGen.convertType(globalOp.getType(), fileAttr, scope,
                                     globalOp.getLoc());

flang/test/Transforms/debug-module-1.f90

@@ -36,4 +36,4 @@ end program test
 
 ! CHECK-DAG: loc(fused<#[[GLR]]>[#[[LOC1]]])
 ! CHECK-DAG: loc(fused<#[[GLI]]>[#[[LOC2]]])
-! CHECK-DAG: loc(fused<#[[TEST]]>[#[[LOC3]]])
+! CHECK-DAG: loc(fused<#[[TEST]]>[#[[LOC3]]])