On Wed, Jul 3, 2019 at 8:52 PM Nick Desaulniers <ndesaulni...@google.com> wrote: > > On Wed, Jul 3, 2019 at 9:15 AM Eric Biggers <ebigg...@kernel.org> wrote: > > Sorry, I am still confused. Are you saying that something still needs to be > > fixed in the kernel code, and if so, why? To reiterate, the byteshift_table > > doesn't actually *need* any particular alignment. Would it avoid the > > confusion > > if I changed it to no alignment? Or is there some section merging related > > reason it actually needs to be 32? > > Looks like the section merging of similarly named sections of > differing alignment in LLD just got reverted: > https://bugs.llvm.org/show_bug.cgi?id=42289#c8 > I wasn't able to find any documentation that said alignment must match > entity size, but if there's not a functional reason for them to differ > then it seems like LLD need not even support such a particularly > non-common case. >
My primary goal was to get rid of these sysfs warnings when building with clang-9 and linking with lld-9 (details see [0]). "clang-9" and "lld-9" are snapshot versions (details see [1]). - Sedat - [0] https://github.com/ClangBuiltLinux/linux/issues/431 [1] https://github.com/ClangBuiltLinux/linux/issues/431#issuecomment-508354865
