[PATCH] D127581: [ELF] Relax R_RISCV_ALIGN

[Fangrui Song via Phabricator via llvm-commits]

MaskRay 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;
----------------
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.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D127581