[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