Skip to content
This repository has been archived by the owner on Apr 11, 2024. It is now read-only.

去除未使用且无用的重定位类型: R_LARCH_ADD24 + R_LARCH_SUB24 #51

Open
mips-hp opened this issue May 13, 2022 · 4 comments

Comments

@mips-hp
Copy link

mips-hp commented May 13, 2022

问题

在修正一个重定位相关的问题的过程中,发现LA定义了 R_LARCH_ADD24和R_LARCH_SUB24类型的重定位,相比R_LARCH_{ADD,SUB}{8,16,32,64}(这些重定位的处理有另外的问题),.eh_frame/DWARF等元数据中的地址或者偏移使用24bit宽度的重定位会让数据对齐处理非常复杂,LA ABI也未定义uint24/int24这样的数据类型;

状态

R_LARCH_ADD24 / R_LARCH_SUB24 目前没有被使用;

修改意见

@xen0n
Copy link
Contributor

xen0n commented May 13, 2022

链接裂了(指向的地址和链接文字不一样),不过意见比较有道理,等回复了

@ChenghuaXu
Copy link
Contributor

这个也关了吧,旧的类型里面不动了,加到新的类型里面去了。

@xen0n
Copy link
Contributor

xen0n commented Aug 16, 2022

他这是删旧的啊,我是建议该删的还是要删,没用的玩意留着他干嘛

@yetist
Copy link
Contributor

yetist commented Nov 9, 2022

没有使用的话,就删掉吧

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants