Deprecate R_RISCV_RVC_LUI? about riscv-elf-psabi-doc HOT 4 CLOSED

I guess that's more like an intermediate relocation used for the linker relaxation process, like R_RISCV_GPREL_[IS] relative relocation ( #205), because I also don't believe that's can be useful to let user use c.lui with this relocation directly, so we should consider deprecate and use this relocation as internal intermediate relocation only.

@Nelson1225 @MaskRay @jrtc27 any thought for this?

from riscv-elf-psabi-doc.

If there's no assembly syntax to emit that relocation and the relocation is just a binutils internal relocation, we probably should remove it altogether from the psABI. It should be safe to do so.

from riscv-elf-psabi-doc.

I guess that's more like an intermediate relocation used for the linker relaxation process, like R_RISCV_GPREL_[IS] relative relocation ( #205), because I also don't believe that's can be useful to let user use c.lui with this relocation directly, so we should consider deprecate and use this relocation as internal intermediate relocation only.

Yap, user cannot use this relocation directly in GNU assembler. We only accept immediate value in c.lui in gas, without any modifiers. However, it's also really rare to see this relaxation in ld, since we should reserve the maxpagesize and commongaesize for data segment alignment (pr27566, from Jim's comment). The pagesize usually occupied most of the rvc_lui space...

If there's no assembly syntax to emit that relocation and the relocation is just a binutils internal relocation, we probably should remove it altogether from the psABI. It should be safe to do so.

Looks good to me :-)

from riscv-elf-psabi-doc.

Removing R_RISCV_RVC_LUI looks good, like we removed R_RISCV_GNU_VTINHERIT, R_RISCV_GNU_VTENTRY, R_RISCV_GPREL_I, R_RISCV_GPREL_S that were not produced by assemblers.

from riscv-elf-psabi-doc.