[PATCH] D74023: [RISCV] ELF attribute section for RISC-V

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Feb 29 09:58:22 PST 2020


MaskRay added a comment.

In lld, SHT_ARM_ATTRIBUTES sections are deduplicated, thus `test/ELF/arm-attributes.s`. No code deduplicate SHT_RISCV_ATTRIBUTES, thus I still suggest we delete the lld test change.

Landing D75015 <https://reviews.llvm.org/D75015> first can reduce changes in this patch....



================
Comment at: lld/test/ELF/Inputs/riscv-attributes1.s:2
+// Input that generates an object with a populated SHT_RISCV_ATTRIBUTES section.
+.text
+.attribute Tag_stack_align, 16
----------------
`.text` can be deleted


================
Comment at: lld/test/ELF/riscv-attributes.s:43
+_start:
+  .globl func
+  call func
----------------
HsiangKai wrote:
> MaskRay wrote:
> > To disable a missing entry warning:
> > 
> > ```
> > .globl _start
> > _start:
> > ```
> > 
> > is sufficient. 
> I want to call another function `func` located in another file `riscv-attributes1.s`.
`ld.lld %t1.rv32.o %t2.rv32.o` links both object files. It is not necessary to add a `call`.



Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D74023





More information about the llvm-commits mailing list