[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