[PATCH] D57940: Refactor RelocVisitor and fix computation of SHT_RELA-typed relocation entries

Thu Mar 21 17:25:10 PDT 2019

arichardson added a comment.

In D57940#1438813 <https://reviews.llvm.org/D57940#1438813>, @MaskRay wrote:

> dispatch using Config->IsRela

The value of `getAddend<ELFT>(Rel);` now only seems to be in `Ref.getRawDataRefImpl().p` now. Shouldn't the rel case also be adding the implicit addend? I'm not sure I'm reading the code currently but it looks to me like REL no longer gets the addend added.

Repository:
  rLLD LLVM Linker

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

https://reviews.llvm.org/D57940