[PATCH] D105217: [LLD] Adding support for RELA for CG Profile.

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 9 14:11:03 PDT 2021


MaskRay added inline comments.


================
Comment at: lld/test/ELF/cgprofile-rela.test:32
+    Flags: [ SHF_ALLOC, SHF_EXECINSTR ]
+    AddressAlign: 0x10
+    Content: 0000
----------------
Delete unused `AddressAlign: 0x10`


================
Comment at: lld/test/ELF/cgprofile-rela.test:72
+    Relocations:
+      - Symbol: A
+        Type:   R_X86_64_NONE
----------------
Add `Offset: 0x0`


================
Comment at: lld/test/ELF/cgprofile-rela.test:120
+    Section: .text.A
+  - Name: .text.D
+    Type: STT_SECTION
----------------
Delete STT_SECTION symbols not referenced by relocations.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D105217



More information about the llvm-commits mailing list