[llvm] [llvm-profgen] Loading binary functions from .symtab when DWARF info is incomplete (PR #163654)

via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 16 22:35:34 PDT 2025


================
@@ -820,6 +822,48 @@ void ProfiledBinary::populateSymbolAddressList(const ObjectFile *Obj) {
   }
 }
 
+void ProfiledBinary::populateSymbolsFromElf(const ObjectFile *Obj) {
----------------
HighW4y2H3ll wrote:

Thanks for the feedbacks! Let me fix up the non-ELF compatibility first. I added another `SymbolRef::getSize()` API to take care of that. Not sure if this is the best way to abstract things, but I'm thinking of following with another NFC patch that renames `ObjectFile::getCommonSymbolSizeImpl` to `ObjectFile::getSymbolSizeImpl`. (so this API no longer attaches to the `SymbolRef::SF_Common` flag?)

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


More information about the llvm-commits mailing list