[PATCH] D150722: [lld][RISCV] Handle relaxation reductions of more than 65536 bytes

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 16 14:28:27 PDT 2023


MaskRay accepted this revision.
MaskRay added a comment.
This revision is now accepted and ready to land.

IIRC @luismarques mentions that with LTO we may see a `bytesDropped` overflow. OK, I don't object to the idea that we pay some memory overhead to make this work.

In D150722#4347763 <https://reviews.llvm.org/D150722#4347763>, @jrtc27 wrote:

> Current code models are such that you can’t have more than 2 GiB of code, so 32-bit seems fine I think?

The `fatal("section size decrease is too large: " + Twine(delta));` is definitely useless for regular code. `uint64_t delta = 0;` may catch some people's linker stress tests...


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D150722



More information about the llvm-commits mailing list