forked from OSchip/llvm-project
[JITLink][AArch64] Rename PointerToGOT and fix typo.
PointerToGOT lowering was accidentally changed from Delta32 to Delta64 in
db37225803
. 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).
This commit is contained in:
parent
51c705fbe5
commit
7ac7837080
|
@ -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:
|
||||
|
|
|
@ -425,7 +425,7 @@ private:
|
|||
else
|
||||
return TargetSymbolOrErr.takeError();
|
||||
|
||||
Kind = aarch64::PointerToGOT;
|
||||
Kind = aarch64::Delta32ToGOT;
|
||||
break;
|
||||
case MachODelta32:
|
||||
case MachODelta64: {
|
||||
|
|
|
@ -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:
|
||||
|
|
Loading…
Reference in New Issue