[PATCH] D127581: [ELF] Relax R_RISCV_ALIGN

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

luismarques added a subscriber: tmsriram.
luismarques added inline comments.


================
Comment at: lld/ELF/Arch/RISCV.cpp:599
+  if (!isUInt<16>(delta))
+    fatal("section size decrease is too large");
+  sec.bytesDropped = delta;
----------------
MaskRay wrote:
> luismarques wrote:
> > @MaskRay I ran into this error when building LLVM with LLD in a RISC-V host. I guess we actually need an int32?
> This will increase the size of InputSection which we should try to avoid (memory usage increase). I am on a trip so cannot investigate it closely.
> 
> It will help if you can ask the author of `--optimize-bb-jumps` whether it is still used. Removing `nopFiller` will make room for delta.
> It will help if you can ask the author of `--optimize-bb-jumps` whether it is still used. Removing `nopFiller` will make room for delta.

@tmsriram any comments?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D127581