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

[Ash Dobrescu via llvm-commits]

================
@@ -2439,6 +2439,33 @@ void RewriteInstance::readDynamicRelocations(const SectionRef &Section,
     if (Symbol)
       SymbolIndex[Symbol] = getRelocationSymbol(InputFile, Rel);
 
+    const uint64_t ReferencedAddress = SymbolAddress + Addend;
+    BinaryFunction *Func =
+        BC->getBinaryFunctionContainingAddress(ReferencedAddress);
+
+    if (Relocation::isRelative(RType) && SymbolAddress == 0) {
+      if (Func) {
+        if (!Func->isInConstantIsland(ReferencedAddress)) {
+          if (const uint64_t ReferenceOffset =
+                  ReferencedAddress - Func->getAddress()) {
+            Func->addEntryPointAtOffset(ReferenceOffset);
+          } else if (ReferencedAddress < Func->getAddress()) {
----------------
Rin18 wrote:

I think we can hit the case where the contained address is the same as the function address, leading `ReferenceOffset` to be 0. But the else if condition is redundant you're right. I have removed it.

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