From e5fc826f8876fc10308745833d94dcd428ffcb83 Mon Sep 17 00:00:00 2001 From: Lang Hames Date: Thu, 17 Jul 2014 23:11:30 +0000 Subject: [PATCH] [MCJIT] Fix the alignment requirements for ARM and AArch64 which were mistakenly relaxed in the big RuntimeDyldMachO cleanup of r213293. No test case yet - this was found via inspection and there's no easy way to test GOT alignment in RuntimeDyldChecker at the moment. I'm working on adding support for this now, and hope to have a test case for this soon. llvm-svn: 213331 --- llvm/lib/ExecutionEngine/RuntimeDyld/Targets/RuntimeDyldMachOAArch64.h | 2 +- llvm/lib/ExecutionEngine/RuntimeDyld/Targets/RuntimeDyldMachOARM.h | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/llvm/lib/ExecutionEngine/RuntimeDyld/Targets/RuntimeDyldMachOAArch64.h b/llvm/lib/ExecutionEngine/RuntimeDyld/Targets/RuntimeDyldMachOAArch64.h index c9fcede..9f48e44 100644 --- a/llvm/lib/ExecutionEngine/RuntimeDyld/Targets/RuntimeDyldMachOAArch64.h +++ b/llvm/lib/ExecutionEngine/RuntimeDyld/Targets/RuntimeDyldMachOAArch64.h @@ -24,7 +24,7 @@ public: unsigned getMaxStubSize() override { return 8; } - unsigned getStubAlignment() override { return 1; } + unsigned getStubAlignment() override { return 8; } relocation_iterator processRelocationRef(unsigned SectionID, relocation_iterator RelI, diff --git a/llvm/lib/ExecutionEngine/RuntimeDyld/Targets/RuntimeDyldMachOARM.h b/llvm/lib/ExecutionEngine/RuntimeDyld/Targets/RuntimeDyldMachOARM.h index 7e14992..1de9942 100644 --- a/llvm/lib/ExecutionEngine/RuntimeDyld/Targets/RuntimeDyldMachOARM.h +++ b/llvm/lib/ExecutionEngine/RuntimeDyld/Targets/RuntimeDyldMachOARM.h @@ -23,7 +23,7 @@ public: unsigned getMaxStubSize() override { return 8; } - unsigned getStubAlignment() override { return 1; } + unsigned getStubAlignment() override { return 4; } relocation_iterator processRelocationRef(unsigned SectionID, relocation_iterator RelI, -- 2.7.4