[llvm-branch-commits] [lldb] [LLDB] Support Re-export Symbol for FunctionCall (PR #208998)

[via llvm-branch-commits]

================
@@ -33,6 +33,30 @@
 using namespace lldb;
 using namespace lldb_private;
 
+/// Pick a call target from \p sc_list.  Re-exported symbols have no code of
+/// their own and are resolved through the library that actually defines
+/// them, mirroring what the dynamic linker does.
+static Address GetCallableAddress(Target &target,
----------------
aokblast wrote:

All of the caller implement their for loop to handle this. It is the only caller of ResolveReExportedSymbol that doesn't have for loop. I agree with you that we should unify this. But it should be a separate patch I think.

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