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

Fangrui Song via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 27 18:40:10 PDT 2023


MaskRay wrote:

There are certainly places where alignment directives are like mandatory, but I feel that reporting an error would make lld not usable for many many benign cases. Consider the scenario that if your crt1.o or crtbeginS.o have R_LARCH_ALIGN.

This interop is exactly why I warned in https://github.com/loongson/LoongArch-Documentation/pull/77 . At the very least, binutils should not enable `-mrelax` by default. Reporting the error is now no better than relaxing R_LARCH_ALIGN.

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


More information about the llvm-commits mailing list