[JITLink][AArch64] Rename PointerToGOT and fix typo.
authorLang Hames <lhames@gmail.com>
Sat, 9 Jul 2022 01:55:43 +0000 (18:55 -0700)
committerLang Hames <lhames@gmail.com>
Sat, 9 Jul 2022 15:09:23 +0000 (08:09 -0700)
PointerToGOT lowering was accidentally changed from Delta32 to Delta64 in
db3722580335c. This patch moves it back to Delta32 and renames the generic
aarch64 edge to Delta32ToGOT to avoid the ambiguity.

No test case yet -- I haven't figured out how to write a succinct test case
(this typically appears in CIEs in eh-frames).

llvm/include/llvm/ExecutionEngine/JITLink/aarch64.h
llvm/lib/ExecutionEngine/JITLink/MachO_arm64.cpp
llvm/lib/ExecutionEngine/JITLink/aarch64.cpp

index 7262601..f2c3fba 100644 (file)
@@ -35,7 +35,7 @@ enum EdgeKind_aarch64 : Edge::Kind {
   TLVPageOffset12,
   TLSDescPage21,
   TLSDescPageOffset12,
-  PointerToGOT,
+  Delta32ToGOT,
   PairedAddend,
   LDRLiteral19,
   Delta32,
@@ -230,7 +230,7 @@ inline Error applyFixup(LinkGraph &G, Block &B, const Edge &E) {
   case TLSDescPageOffset12:
   case GOTPage21:
   case GOTPageOffset12:
-  case PointerToGOT: {
+  case Delta32ToGOT: {
     return make_error<JITLinkError>(
         "In graph " + G.getName() + ", section " + B.getSection().getName() +
         "GOT/TLV edge kinds not lowered: " + getEdgeKindName(E.getKind()));
@@ -277,8 +277,8 @@ public:
              "RawInstr isn't a 64-bit LDR immediate");
       break;
     }
-    case aarch64::PointerToGOT: {
-      KindToSet = aarch64::Delta64;
+    case aarch64::Delta32ToGOT: {
+      KindToSet = aarch64::Delta32;
       break;
     }
     default:
index dd50314..0419431 100644 (file)
@@ -425,7 +425,7 @@ private:
           else
             return TargetSymbolOrErr.takeError();
 
-          Kind = aarch64::PointerToGOT;
+          Kind = aarch64::Delta32ToGOT;
           break;
         case MachODelta32:
         case MachODelta64: {
index e486bef..9ecc71d 100644 (file)
@@ -52,8 +52,8 @@ const char *getEdgeKindName(Edge::Kind R) {
     return "TLSDescPage21";
   case TLSDescPageOffset12:
     return "TLSDescPageOffset12";
-  case PointerToGOT:
-    return "PointerToGOT";
+  case Delta32ToGOT:
+    return "Delta32ToGOT";
   case PairedAddend:
     return "PairedAddend";
   case LDRLiteral19: