gbenyei added a comment. In D128612#3617955 <https://reviews.llvm.org/D128612#3617955>, @MaskRay wrote:
> lld/ELF change should be dropped from this change. Don't use > `config->endianness`. > I feel sad that for little-endian users who don't use big-endian, every write > now is slightly slower due to a check ;-) Hi, I'm not sure I get it. How will we have a fully functional toolchain, if I don't implement the lld/ELF part? In LLVM, unlike in GCC, target related decisions happen in runtime. I think it's a high level design decision. While I can understand the pain of LE developers getting a slightly slower linker due to endianness checking, I sure will feel the pain of a BE developer not having a linker... Please explain why I shouldn't use `config->endianness`? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D128612/new/ https://reviews.llvm.org/D128612 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
