[PATCH] D105214: [ARM] RELA relocations for 32bit ARM ignore the addend.

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 30 15:10:46 PDT 2021


MaskRay added inline comments.


================
Comment at: llvm/lib/Object/RelocationResolver.cpp:314
 
 static uint64_t resolveARM(uint64_t Type, uint64_t Offset, uint64_t S,
+                           uint64_t LocData, int64_t Addend) {
----------------
The code is written in the assumption that targets preferring RELA always have Addend while targets preferring REL always have LocData.

Are you trying to add RELA support to ARM? I think it is unpopular but maybe you have specific needs which are not specified.

No objection, but a comment is needed.


================
Comment at: llvm/test/DebugInfo/ARM/dwarfdump-rela.yaml:34
+    EntSize:         0x0
+  - Name:            .rel.debug_info
+    Type:            SHT_RELA
----------------
rela


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

https://reviews.llvm.org/D105214



More information about the llvm-commits mailing list