[PATCH] D143345: [RFC][RISCV] Don't disassemble `addi`s with relocations as `mv`s
Jessica Clarke via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Feb 15 17:37:18 PST 2023
jrtc27 added a comment.
In D143345#4130588 <https://reviews.llvm.org/D143345#4130588>, @luismarques wrote:
> In D143345#4130585 <https://reviews.llvm.org/D143345#4130585>, @craig.topper wrote:
>
>> It'd be really cool if we could use the relocation to print %pcrel_lo(sym) instead of 0. But I don't know how much work that would be.
>
> I can work on that if there is agreement that we should go in that direction. Makes sense to me, but I wonder if some people will push back against that, regardless of implementation concerns.
>From my perspective that would be cool and useful for all targets. I can’t think of a reason why one wouldn’t prefer that, beyond the complexity of having to synthesise labels when the assembler has converted them to section-relative.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D143345/new/
https://reviews.llvm.org/D143345
More information about the llvm-commits
mailing list