[lld] [llvm] [RISCV] Fix alignment when mixing rvc/norvc relax/norelax code (PR #150159)

Sam Elliott via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 1 11:39:06 PDT 2025


lenary wrote:

I don't know of the status of this vs #150816. I presume we're only getting one of these, not both.

In terms of naming, I would prefer we don't reference rvc, and instead we reference this in terms of the minimal `IALIGN` - which for any section will be 4 until someone enables Zca at which point it will drop to 2. I think conceptually that's a bit more specific than noting something about "rvc" when the check is for zca, a subset of rvc. I'm happy to accept the code would be equivalent, but I think naming in terms of `IALIGN` would help clarify what's really going on.



https://github.com/llvm/llvm-project/pull/150159


More information about the llvm-commits mailing list