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

[Ash Dobrescu 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);
----------------
Rin18 wrote:

>The question remains what happens if we have dynamic relocations of non-relative type against text

In that situation this change has no effect and we can revert back to the current status quo: using `--skip-funcs` to deal with errors.
>We should handle it even if we don't anticipate it to happen in practice.

Since this patch doesn't introduce errors in the case of dynamic relocations of non-relative type, I think we should handle those cases in a separate patch.

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