[PATCH] D100835: [WIP][LLD][RISCV] Linker Relaxation

[Luís Marques via Phabricator via llvm-commits]

luismarques added a comment.

This was discussed in the RISC-V sync-up call. As suggested by Alex, I'm sharing in writing here some of the things I mentioned or were discussed in the call:

- I found a linking failure issue with this patch. I hadn't yet reported that here because I am still in the process of reducing and analyzing the problematic case.
- We touched on the topic of relaxation performance and I shared this graph of my benchmarking results of LLD with this patch compared with BFD: F22778264: lld-relax-perf.png <https://reviews.llvm.org/F22778264>
- @gkm reported that he was also analyzing/working on this. If anybody else is planning to work on this it would be good to coordinate with him and the rest of the community, to avoid duplicate work.



================
Comment at: lld/ELF/Arch/RISCV.cpp:476
+
+// Relax R_RISCV_CALL to jal or c.jal.
+//
----------------
This is also called for R_RISCV_CALL_PLT


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

https://reviews.llvm.org/D100835