[PATCH] D65865: [ELF][X86] Allow PT_LOAD to have overlapping p_offset ranges on EM_386

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 8 02:01:45 PDT 2019


MaskRay added inline comments.


================
Comment at: test/ELF/plt-i686.s:123
 // 0x3000 + got.plt.reserved(12) = 0x300C
 // 0x3000 + got.plt.reserved(12) + 4 = 0x3010
 // CHECKSHARED:        Relocations [
----------------
grimar wrote:
> These are not valid anymore.
> 
> The others in this file too. Though they seems became broken before this change.
I'll delete

```
// Unfortunately FileCheck can't do math, so we have to check for explicit
// values:

// 16 is the size of PLT[0]
// (0x401010 + 16) - (0x401000 + 1) - 4 = 27
// (0x401010 + 16) - (0x401005 + 1) - 4 = 22
// (0x401020 + 16) - (0x40100a + 1) - 4 = 33
```

Hopefully FileCheck will soon learn how to do math. (There is a patch series but it hasn't fully landed yet.)

jmp instructions are annotated with PLT entries so the math is no longer necessary.


Repository:
  rLLD LLVM Linker

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D65865/new/

https://reviews.llvm.org/D65865





More information about the llvm-commits mailing list