[PATCH] D144617: [AVR][MC] Add ELF flag 'EF_AVR_LINKRELAX_PREPARED' to OBJ files

Thu Feb 23 00:08:22 PST 2023

benshi001 added a comment.

In D144617#4146563 <https://reviews.llvm.org/D144617#4146563>, @MaskRay wrote:

> I'm concerned of this change. Do you intend to add linker relaxation to lld/ELF's AVR port? Implementing features is indeed fun but I am unsure whether the use case has a sufficient demand of the feature.
> RISC-V linker relaxation has greatly increased code complexity and LoongArch folks want to copy that. I don't want to see a third arch which may use a non-recommended way implementing linker relaxation.

I have no plan for adding linker relaxation to lld/ELF's AVR port, at least, you can deny that if you think it introduces complexity to lld.

However, avr-ld indeed has already implemented linker relaxation, my current patch can make use of it, and clang+avr-ld can generate shorter assembly.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D144617