[lld] [lld][LoongArch] Print error when encountering R_LARCH_ALIGN (PR #67424)

Fangrui Song via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 26 19:04:27 PDT 2023


MaskRay wrote:

Err, binutils defaulting to `-mrelax` and copying `R_RISCV_ALIGN` is unfortunate. I've mentioned it could be probelamtic: https://github.com/loongson/LoongArch-Documentation/pull/77 

For many align directives, not respecting the required alignment isn't exactly an issue. Printing an error would likely be more disruptive. Perhaps the choice is just to silently ignore.

Now with GitHub pull requests, it's difficult to become a blocking reviewer, but I'll request changes for now.

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


More information about the llvm-commits mailing list