On Sat, Oct 26, 2024 at 02:52:52PM +0000, Koakuma wrote: > Masahiro Yamada <masahiroy@xxxxxxxxxx> wrote: > > I think this should be documented (required LLVM version and > > the supported build command), > > otherwise people cannot test this patch. > > Nathan Chancellor <nathan@xxxxxxxxxx> wrote: > > I am not sure that there is a super concise way to describe for > > Documentation/kbuild/llvm.rst that sparc currently requires 'CC=clang > > LLVM_IAS=0' along with a build of clang from the main branch of > > llvm-project to work properly. > > So about this, as a middle ground, would it be okay if I put > > ``CC=clang LLVM_IAS=0`` (LLVM >= 20) > > In the documentation, in a similar manner to the s390x entry? > I know that LLVM 20 is still a couple months away but those commits will > likely be released with that version, and since it also tells people > to not use a version that is too old, I think it should be okay (?) Yes, I think that would be reasonable. Cheers, Nathan
