[llvm] [gSYM] Add support merged functions in gSYM format (PR #101604)

Greg Clayton via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 2 13:15:07 PDT 2024


================
@@ -357,6 +357,11 @@ static llvm::Error handleObjectFile(ObjectFile &Obj, const std::string &OutFile,
   if (auto Err = DT.convert(ThreadCount, Out))
     return Err;
 
+  // Organize overlapping functions as children of top-level functions. Do this
+  // right after loading the DWARF data so we don't have to deal with functions
+  // from the symbol table.
----------------
clayborg wrote:

This comment makes it seam like we will merge any functions with overlapping ranges. Maybe some different wording here?:
```
// Merge functions with identical address ranges as merged functions in the 
// first FunctionInfo with that address range. Do this right after loading the 
// DWARF data so we don't have to deal with functions from the symbol table.
```


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


More information about the llvm-commits mailing list