[llvm] [BOLT] Support computed goto and allow map addrs inside functions (PR #120267)

[Maksim Panchenko via llvm-commits]

================
@@ -2439,6 +2439,26 @@ void RewriteInstance::readDynamicRelocations(const SectionRef &Section,
     if (Symbol)
       SymbolIndex[Symbol] = getRelocationSymbol(InputFile, Rel);
 
+    const uint64_t ReferencedAddress = SymbolAddress + Addend;
+    BinaryFunction *Func =
+        BC->getBinaryFunctionContainingAddress(ReferencedAddress);
----------------
maksfb wrote:

Cool. If `R_*_RELATIVE` is the only relocation type you want to handle here, that should work. The dynamic linker will ignore they symbol even if it's present.

The question remains what happens if we have dynamic relocations of non-relative type against text. We should handle it even if we don't anticipate it to happen in practice.

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