[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