[llvm-gsymutil] Fix broken tests

[alx32]

Recently #120991 broke a couple of tests.
Also macho-merged-funcs-dwarf.yaml was already flaky due to some non-determinism issues.

Fixing the previous code to not break tests and modifying macho-merged-funcs-dwarf.yaml to fix the non-determinism (which will be resolved later).

[llvmbot]

@llvm/pr-subscribers-debuginfo

Author: None (alx32)

Changes

Recently #120991 broke a couple of tests.
Also macho-merged-funcs-dwarf.yaml was already flaky due to some non-determinism issues.

Fixing the previous code to not break tests and modifying macho-merged-funcs-dwarf.yaml to fix the non-determinism (which will be resolved later).

---

Full diff: https://github.com/llvm/llvm-project/pull/121837.diff

2 Files Affected:

• (modified) llvm/test/tools/llvm-gsymutil/ARM_AArch64/macho-merged-funcs-dwarf.yaml (+3-3)
• (modified) llvm/tools/llvm-gsymutil/llvm-gsymutil.cpp (+14-16)

diff --git a/llvm/test/tools/llvm-gsymutil/ARM_AArch64/macho-merged-funcs-dwarf.yaml b/llvm/test/tools/llvm-gsymutil/ARM_AArch64/macho-merged-funcs-dwarf.yaml
index bcd3d7847da459..31dfdc8b8983be 100644
--- a/llvm/test/tools/llvm-gsymutil/ARM_AArch64/macho-merged-funcs-dwarf.yaml
+++ b/llvm/test/tools/llvm-gsymutil/ARM_AArch64/macho-merged-funcs-dwarf.yaml
@@ -69,9 +69,9 @@
 # RUN: llvm-gsymutil --verify %t.keep.gSYM --address 0x248 | FileCheck --check-prefix=CHECK-NORMAL-LOOKUP %s
  
 # CHECK-MERGED-LOOKUP: Found 3 functions at address 0x0000000000000248:
-# CHECK-MERGED-LOOKUP-NEXT:       0x0000000000000248: my_func_02 @ /tmp/test_gsym_yaml/out/file_02.cpp:5
-# CHECK-MERGED-LOOKUP-NEXT-NEXT:  0x0000000000000248: my_func_01 @ /tmp/test_gsym_yaml/out/file_01.cpp:5
-# CHECK-MERGED-LOOKUP-NEXT-NEXT:  0x0000000000000248: my_func_03 @ /tmp/test_gsym_yaml/out/file_03.cpp:5
+# CHECK-MERGED-LOOKUP-NEXT:       0x0000000000000248: my_func_0{{[1-3]}} @ /tmp/test_gsym_yaml/out/file_0{{[1-3]}}.cpp:5
+# CHECK-MERGED-LOOKUP-NEXT-NEXT:  0x0000000000000248: my_func_0{{[1-3]}} @ /tmp/test_gsym_yaml/out/file_0{{[1-3]}}.cpp:5
+# CHECK-MERGED-LOOKUP-NEXT-NEXT:  0x0000000000000248: my_func_0{{[1-3]}} @ /tmp/test_gsym_yaml/out/file_0{{[1-3]}}.cpp:5
  
 # CHECK-NORMAL-LOOKUP: 0x0000000000000248: my_func_01 @ /tmp/test_gsym_yaml/out/file_01.cpp:5
 
diff --git a/llvm/tools/llvm-gsymutil/llvm-gsymutil.cpp b/llvm/tools/llvm-gsymutil/llvm-gsymutil.cpp
index e6562b9ebf4049..654da68bb69600 100644
--- a/llvm/tools/llvm-gsymutil/llvm-gsymutil.cpp
+++ b/llvm/tools/llvm-gsymutil/llvm-gsymutil.cpp
@@ -508,11 +508,6 @@ static llvm::Error convertFileToGSYM(OutputAggregator &Out) {
 }
 
 static void doLookup(GsymReader &Gsym, uint64_t Addr, raw_ostream &OS) {
-  auto logError = [Addr, &OS](Error E) {
-    OS << HEX64(Addr) << ": ";
-    logAllUnhandledErrors(std::move(E), OS, "error: ");
-  };
-
   if (UseMergedFunctions) {
     if (auto Results = Gsym.lookupAll(Addr)) {
       OS << "Found " << Results->size() << " functions at address "
@@ -526,20 +521,23 @@ static void doLookup(GsymReader &Gsym, uint64_t Addr, raw_ostream &OS) {
     }
   } else { /* UseMergedFunctions == false */
     if (auto Result = Gsym.lookup(Addr)) {
+      // If verbose is enabled dump the full function info for the address.
+      if (Verbose) {
+        if (auto FI = Gsym.getFunctionInfo(Addr)) {
+          OS << "FunctionInfo for " << HEX64(Addr) << ":\n";
+          Gsym.dump(OS, *FI);
+          OS << "\nLookupResult for " << HEX64(Addr) << ":\n";
+        }
+      }
       OS << Result.get();
     } else {
-      logError(Result.takeError());
-      return;
-    }
-  }
-
-  if (Verbose) {
-    if (auto FI = Gsym.getFunctionInfo(Addr)) {
-      OS << "FunctionInfo for " << HEX64(Addr) << ":\n";
-      Gsym.dump(OS, *FI);
-      OS << "\nLookupResult for " << HEX64(Addr) << ":\n";
+      if (Verbose)
+        OS << "\nLookupResult for " << HEX64(Addr) << ":\n";
+      OS << HEX64(Addr) << ": ";
+      logAllUnhandledErrors(Result.takeError(), OS, "error: ");
     }
-    OS << "\n";
+    if (Verbose)
+      OS << "\n";
   }
 }
 

[ellishg]

Can you use CHECK-MERGED-LOOKUP-DAG instead?

[alx32]

Will keep in mind when addressing the non-determinism in the future (this PR is already merged).