[all-commits] [llvm/llvm-project] e258ad: [Object/ELF] - Fix a position calculation expressi...

Georgii Rymar via All-commits all-commits at lists.llvm.org
Thu Mar 5 01:49:47 PST 2020


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: e258ad51293f9c363aee8f9b990c6502dd4d14ac
      https://github.com/llvm/llvm-project/commit/e258ad51293f9c363aee8f9b990c6502dd4d14ac
  Author: Georgii Rymar <grimar at accesssoftek.com>
  Date:   2020-03-05 (Thu, 05 Mar 2020)

  Changed paths:
    M llvm/include/llvm/Object/ELF.h

  Log Message:
  -----------
  [Object/ELF] - Fix a position calculation expression in ELFFile<ELFT>::getEntry().

It fixes now what 1c991f907a43d7a56e82dd67a76514843841ed9a tried to fix.
(A test case failture on 32-bit Arch Linux)

On 32-bit hosts it still fails (because it truncates the `Pos` value to 32 bits).
It seems happens because of `sizeof` that returns `size_t`, which has a
different size on 32/64 bits hosts.

I've tested on a 32-bit host and verified that relocation-errors.test test and
other LLVM tools tests pass now.




More information about the All-commits mailing list