diff --git a/lib/MC/ELFObjectWriter.cpp b/lib/MC/ELFObjectWriter.cpp index 35f029d2fa1..ebbba1327dd 100644 --- a/lib/MC/ELFObjectWriter.cpp +++ b/lib/MC/ELFObjectWriter.cpp @@ -541,10 +541,13 @@ void ELFObjectWriterImpl::RecordRelocation(const MCAssembler &Asm, case X86::reloc_pcrel_4byte: case FK_Data_4: // check that the offset fits within a signed long - if (isInt<32>(Target.getConstant())) + if (Target.getConstant() < 0) { + assert(isInt<32>(Target.getConstant())); Type = ELF::R_X86_64_32S; - else + } else { + assert(isUInt<32>(Target.getConstant())); Type = ELF::R_X86_64_32; + } break; case FK_Data_2: Type = ELF::R_X86_64_16; break; case X86::reloc_pcrel_1byte: diff --git a/test/MC/ELF/basic-elf.ll b/test/MC/ELF/basic-elf.ll index abdce7e3589..bc9ec131860 100644 --- a/test/MC/ELF/basic-elf.ll +++ b/test/MC/ELF/basic-elf.ll @@ -72,7 +72,7 @@ declare i32 @puts(i8* nocapture) nounwind ; 64: ('_relocations', [ ; 64: # Relocation 0 ; 64: (('r_offset', 5) -; 64: ('r_type', 11) +; 64: ('r_type', 10) ; 64: ('r_addend', 0) ; 64: ), ; 64: # Relocation 1 @@ -82,7 +82,7 @@ declare i32 @puts(i8* nocapture) nounwind ; 64: ), ; 64: # Relocation 2 ; 64: (('r_offset', 15) -; 64: ('r_type', 11) +; 64: ('r_type', 10) ; 64: ('r_addend', 6) ; 64: ), ; 64: # Relocation 3 diff --git a/test/MC/ELF/relocation.s b/test/MC/ELF/relocation.s new file mode 100644 index 00000000000..b9b87dd2049 --- /dev/null +++ b/test/MC/ELF/relocation.s @@ -0,0 +1,12 @@ +// RUN: llvm-mc -filetype=obj -triple x86_64-pc-linux-gnu %s -o - | elf-dump --dump-section-data | FileCheck %s + +// Test that we produce a R_X86_64_32. + + .long Lset1 + + +// CHECK: # Relocation 0 +// CHECK-NEXT: (('r_offset', 0) +// CHECK-NEXT: ('r_sym', 4) +// CHECK-NEXT: ('r_type', 10) +// CHECK-NEXT: ('r_addend', 0)