search for: clangbuiltlinux

...0 --:--:-- --:--:-- --:--:-- 345 100 25.7M 0 25.7M 0 0 2263k 0 --:--:-- 0:00:11 --:--:-- 2530k^C I have already built from this git-ref "llvmorg-10.0.1-rc1" successfully. $ which clang-10 /home/dileks/src/llvm-toolchain/install/bin/clang-10 $ clang-10 --version ClangBuiltLinux clang version 10.0.1 (https://github.com/llvm/llvm-project f79cd71e145c6fd005ba4dd1238128dfa0dc2cb6) Target: x86_64-unknown-linux-gnu Thread model: posix InstalledDir: /home/dileks/src/llvm-toolchain/install/bin BTW, I use tc-build script from ClangBuiltLinux to build my llvm-toolchain consisting...

...^~~~~~~~~~~ ~~~~ 2 warnings generated. Commit f7e30f01a9e2 ("cpumask: Add helper cpumask_available()") added cpumask_available to fix warnings of this nature. Use that here so that clang does not warn regardless of CONFIG_CPUMASK_OFFSTACK's value. Link: https://github.com/ClangBuiltLinux/linux/issues/982 Reported-by: Sedat Dilek <sedat.dilek at gmail.com> Signed-off-by: Nathan Chancellor <natechancellor at gmail.com> --- arch/x86/mm/mmio-mod.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/arch/x86/mm/mmio-mod.c b/arch/x86/mm/mmio-mod.c index 1...

Hello, I've been trying to add support to the integrated assembler to handle cases like below foo:instr; .if . - foo; instr; .endif when the dot symbol and foo are assigned to consecutive MCDataFragments <https://github.com/ClangBuiltLinux/linux/issues/744>. I have a patch <https://reviews.llvm.org/D6941> under review. And even though it seems to solve the problem, the concern is it complicates the code and we would like to know if there is more decent way to handle this. Please feel free to share your thoughts. Thanks, Jia...

...RSION "${LLVM_VERSION_MAJOR}${LLVM_VERSION_SUFFIX}") +set(LLVM_DYLIB_VERSION "${LLVM_VERSION_MAJOR}") set(LLVM_HAS_GLOBAL_ISEL "ON") # Use the C++ link flags, since they should be a superset of C link flags. - End Of DIFF - With this I now get: $ ./bin/clang-10 -v ClangBuiltLinux clang version 10.0.1rc1 Target: x86_64-unknown-linux-gnu Thread model: posix InstalledDir: /home/dileks/src/llvm-toolchain/build/stage1/./bin Found candidate GCC installation: /usr/lib/gcc/x86_64-linux-gnu/10 Found candidate GCC installation: /usr/lib/gcc/x86_64-linux-gnu/8 Found candidate GCC inst...

...to put a lot of effort into getting a clean tree. So this warning doesn't seem to be worth it. ``` Looks like there's an order of magnitude of `-Wunreachable-code-break` than the other two. We probably should add all 3 to W=2 builds (wrapped in cc-option). I've filed https://github.com/ClangBuiltLinux/linux/issues/1180 to follow up on. -- Thanks, ~Nick Desaulniers

...9; [-Wunused-label] out_del_vqs: ^~~~~~~~~~~~ 1 warning generated. Move the label within the preprocessor block since it is only used when CONFIG_BALLOON_COMPACTION is set. Fixes: 1ad6f58ea936 ("virtio_balloon: Fix memory leaks on errors in virtballoon_probe()") Link: https://github.com/ClangBuiltLinux/linux/issues/886 Signed-off-by: Nathan Chancellor <natechancellor at gmail.com> --- drivers/virtio/virtio_balloon.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/virtio/virtio_balloon.c b/drivers/virtio/virtio_balloon.c index 7bfe365d9372..341458fd95ca 100644 --...

...9; [-Wunused-label] out_del_vqs: ^~~~~~~~~~~~ 1 warning generated. Move the label within the preprocessor block since it is only used when CONFIG_BALLOON_COMPACTION is set. Fixes: 1ad6f58ea936 ("virtio_balloon: Fix memory leaks on errors in virtballoon_probe()") Link: https://github.com/ClangBuiltLinux/linux/issues/886 Signed-off-by: Nathan Chancellor <natechancellor at gmail.com> --- drivers/virtio/virtio_balloon.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/virtio/virtio_balloon.c b/drivers/virtio/virtio_balloon.c index 7bfe365d9372..341458fd95ca 100644 --...

...201014: R_X86_64_32S *ABS*+0x8 201018: 48 89 45 f8 mov %rax,-0x8(%rbp) 20101c: 5d pop %rbp 20101d: c3 retq I'm not sure where to start looking for solving this... Thanks! -Kees [1] https://github.com/ClangBuiltLinux/linux/issues/404 -- Kees Cook

...( ) 1 warning generated. Add the parentheses as it was clearly intended for the ternary condition to be evaluated first. Fixes: 5fd23bd0d739 ("kernel/resource: make release_mem_region_adjustable() never fail") Link: https://github.com/ClangBuiltLinux/linux/issues/1159 Signed-off-by: Nathan Chancellor <natechancellor at gmail.com> --- Presumably, this will be squashed but I included a fixes tag nonetheless. Apologies if this has already been noticed and fixed already, I did not find anything on LKML. kernel/resource.c | 2 +- 1 file cha...

...; > > Commit f7e30f01a9e2 ("cpumask: Add helper cpumask_available()") added > > cpumask_available to fix warnings of this nature. Use that here so that > > clang does not warn regardless of CONFIG_CPUMASK_OFFSTACK's value. > > > > Link: https://github.com/ClangBuiltLinux/linux/issues/982 > > Reported-by: Sedat Dilek <sedat.dilek at gmail.com> > > Signed-off-by: Nathan Chancellor <natechancellor at gmail.com> Thanks for the patch, sorry I'm falling behind on code review! Reviewed-by: Nick Desaulniers <ndesaulniers at google.com> &...

[ Please CC me I ma not subcribed to this mailing-list ] [ CC Tom and Hans as LLVM/stable maintainers ] Hi, I want to understand the version handling in LLVM/Clang/LLD. Normally, I build from "release/10.x" Git branch by using the tool tc-build from ClangBuiltLinux project. With "llvm-10.0.1-rc1" Git tag I was able to setup a llvm-toolchain consisting of the projects "clang" and "lld". A Linux v5.7-rc6 was successfully built. $ cat /proc/version Linux version 5.7.0-rc6-2-amd64-clang (sedat.dilek at gmail.com@iniza) (ClangBuilt...

...o avoid this, use BIT_ULL, which will be an unsigned long long. This matches the size of the features field throughout this driver, which is u64 so there should be no functional change. Fixes: 1a86b377aa21 ("vdpa/mlx5: Add VDPA driver for supported mlx5 devices") Link: https://github.com/ClangBuiltLinux/linux/issues/1140 Signed-off-by: Nathan Chancellor <natechancellor at gmail.com> --- drivers/vdpa/mlx5/net/mlx5_vnet.c | 50 +++++++++++++++---------------- 1 file changed, 25 insertions(+), 25 deletions(-) diff --git a/drivers/vdpa/mlx5/net/mlx5_vnet.c b/drivers/vdpa/mlx5/net/mlx5_vnet.c i...

...o avoid this, use BIT_ULL, which will be an unsigned long long. This matches the size of the features field throughout this driver, which is u64 so there should be no functional change. Fixes: 1a86b377aa21 ("vdpa/mlx5: Add VDPA driver for supported mlx5 devices") Link: https://github.com/ClangBuiltLinux/linux/issues/1140 Signed-off-by: Nathan Chancellor <natechancellor at gmail.com> --- drivers/vdpa/mlx5/net/mlx5_vnet.c | 50 +++++++++++++++---------------- 1 file changed, 25 insertions(+), 25 deletions(-) diff --git a/drivers/vdpa/mlx5/net/mlx5_vnet.c b/drivers/vdpa/mlx5/net/mlx5_vnet.c i...

...warnings generated. > > Commit f7e30f01a9e2 ("cpumask: Add helper cpumask_available()") added > cpumask_available to fix warnings of this nature. Use that here so that > clang does not warn regardless of CONFIG_CPUMASK_OFFSTACK's value. > > Link: https://github.com/ClangBuiltLinux/linux/issues/982 > Reported-by: Sedat Dilek <sedat.dilek at gmail.com> > Signed-off-by: Nathan Chancellor <natechancellor at gmail.com> > --- > Acked-by: Steven Rostedt (VMware) <rostedt at goodmis.org> -- Steve

...2 warnings generated. > > Commit f7e30f01a9e2 ("cpumask: Add helper cpumask_available()") added > cpumask_available to fix warnings of this nature. Use that here so that > clang does not warn regardless of CONFIG_CPUMASK_OFFSTACK's value. > > Link: https://github.com/ClangBuiltLinux/linux/issues/982 > Reported-by: Sedat Dilek <sedat.dilek at gmail.com> > Signed-off-by: Nathan Chancellor <natechancellor at gmail.com> > --- > arch/x86/mm/mmio-mod.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/arch/x86/mm/mmio-mod....

...; > > Commit f7e30f01a9e2 ("cpumask: Add helper cpumask_available()") added > > cpumask_available to fix warnings of this nature. Use that here so that > > clang does not warn regardless of CONFIG_CPUMASK_OFFSTACK's value. > > > > Link: https://github.com/ClangBuiltLinux/linux/issues/982 > > Reported-by: Sedat Dilek <sedat.dilek at gmail.com> > > Signed-off-by: Nathan Chancellor <natechancellor at gmail.com> > > --- > > arch/x86/mm/mmio-mod.c | 4 ++-- > > 1 file changed, 2 insertions(+), 2 deletions(-) > > > >...

...warnings generated. > > Commit f7e30f01a9e2 ("cpumask: Add helper cpumask_available()") added > cpumask_available to fix warnings of this nature. Use that here so that > clang does not warn regardless of CONFIG_CPUMASK_OFFSTACK's value. > > Link: https://github.com/ClangBuiltLinux/linux/issues/982 > Reported-by: Sedat Dilek <sedat.dilek at gmail.com> > Signed-off-by: Nathan Chancellor <natechancellor at gmail.com> > --- > arch/x86/mm/mmio-mod.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/arch/x86/mm/mmio-mod...

...7e30f01a9e2 ("cpumask: Add helper cpumask_available()") added > > > cpumask_available to fix warnings of this nature. Use that here so that > > > clang does not warn regardless of CONFIG_CPUMASK_OFFSTACK's value. > > > > > > Link: https://github.com/ClangBuiltLinux/linux/issues/982 > > > Reported-by: Sedat Dilek <sedat.dilek at gmail.com> > > > Signed-off-by: Nathan Chancellor <natechancellor at gmail.com> > > Thanks for the patch, sorry I'm falling behind on code review! > Reviewed-by: Nick Desaulniers <ndesau...

...on > inl (%dx) > ^ > LLVM ERROR: Error parsing inline asm > > Use the full form of the instruction to fix the build. > > Signed-off-by: Sami Tolvanen <samitolvanen at google.com> Thanks Sami, this looks like it addresses: Link: https://github.com/ClangBuiltLinux/linux/issues/734 Looks like GAS' testsuite has some cases where the second operand is indeed implicitly %eax if unspecified. (This should still be fixed in Clang). Just to triple check that they're equivalent: $ cat inl.s inl (%dx) inl (%dx), %eax $ as inl.s $ objdump -d a.out a.out:...

...where one must guess as to the right result. The compiler will eliminate any actual redundancy. Note that there is already a patch in mainline that addresses almost 40,000 of these issues[6]. [1] commit e2079e93f562c ("kbuild: Do not enable -Wimplicit-fallthrough for clang for now") [2] ClangBuiltLinux#636 [3] https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91432 [4] https://godbolt.org/z/xgkvIh [5] commit a035d552a93b ("Makefile: Globally enable fall-through warning") [6] commit 4169e889e588 ("include: jhash/signal: Fix fall-through warnings for Clang") Thanks -- Gustavo