-
Notifications
You must be signed in to change notification settings - Fork 13.5k
compiler-rt build missing library file #58680
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
Same issue for the following build command:
Failed tests (from
Almost all are due to missing libc++.a, but some are due to failed includes (e.g. |
same issue when running ./test-release.sh -release 17.0.2 -final -triple x86_64-linux-gnu -use-ninja did you find any solution to this? |
I have found a few things, but still no real solution. The missing files seem to be generated from "add_custom_libcxx"
This appears to be similar to the fuzzer tests, which are working fine (and generate the needed files) ie.
I can get the files to be generated (and the tests to pass) by doing
note: The From my understanding, I also noticed that the tests compile fine without the extra args added by
The What is the reasoning for the custom libcxx? Is it fine to use the one from the Phase3 install directory? Hopefully this is of some help. |
@dvyukov Do you have ideas about this? I'm seeing similar errors in the official release builds: https://github.com/llvm/llvm-project/actions/runs/6861861275/job/18658422688 |
@petrhosek I know it's been a while since you worked on this, but do you have any ideas here? |
No, I don't have any ideas. |
I had a look at those logs, not sure if its related, but I am also getting the file differences in hwasan too. I tried doing a phase 4 to see if it was something like #11572 (comment)
Should I open another ticket for this? Do you think its safe to use the output even with the file differences and test failures? |
The test-depends target contained all the dependencies needed to run the runtimes tests, but it was never added as a dependency of check-all. This led to some of the tsan tests to fail, since the custom libcxx build they were using was never built. Fixes llvm#58680
@tstellar I'll try to reproduce this locally and will report back. |
@petrhosek I have a fix here: #72955 |
The test-depends target contained all the dependencies needed to run the runtimes tests, but it was never added as a dependency of check-all. This caused some of the tsan tests to fail, since the custom libcxx build the tests were looking for was never built. Besides the tsan failures, this fixes all the other test failures I was seeing with: cmake -G Ninja -B release-build -S llvm \ -DCMAKE_POSITION_INDEPENDENT_CODE=ON \ -DCMAKE_BUILD_TYPE=Release \ -DLLVM_ENABLE_ASSERTIONS=OFF \ -DLLVM_ENABLE_PROJECTS="clang;lld" \ -DLLVM_ENABLE_RUNTIMES="libcxx;libcxxabi;libunwind;compiler-rt" This is the same configuration the test-release.sh script uses, so I'm hoping this will also fix all the test failures we've been seeing when building the releases. Fixes #58680
The test-depends target contained all the dependencies needed to run the runtimes tests, but it was never added as a dependency of check-all. This caused some of the tsan tests to fail, since the custom libcxx build the tests were looking for was never built. Besides the tsan failures, this fixes all the other test failures I was seeing with: cmake -G Ninja -B release-build -S llvm \ -DCMAKE_POSITION_INDEPENDENT_CODE=ON \ -DCMAKE_BUILD_TYPE=Release \ -DLLVM_ENABLE_ASSERTIONS=OFF \ -DLLVM_ENABLE_PROJECTS="clang;lld" \ -DLLVM_ENABLE_RUNTIMES="libcxx;libcxxabi;libunwind;compiler-rt" This is the same configuration the test-release.sh script uses, so I'm hoping this will also fix all the test failures we've been seeing when building the releases. Fixes #58680 (cherry picked from commit 7f215b1)
This caused some runtimes builds to fail with: error: unknown target 'runtimes-test-depends' See comments on the PR. > The test-depends target contained all the dependencies needed to run the > runtimes tests, but it was never added as a dependency of check-all. > This caused some of the tsan tests to fail, since the custom libcxx > build the tests were looking for was never built. Besides the tsan > failures, this fixes all the other test failures I was seeing with: > cmake -G Ninja -B release-build -S llvm \ > -DCMAKE_POSITION_INDEPENDENT_CODE=ON \ > -DCMAKE_BUILD_TYPE=Release \ > -DLLVM_ENABLE_ASSERTIONS=OFF \ > -DLLVM_ENABLE_PROJECTS="clang;lld" \ > -DLLVM_ENABLE_RUNTIMES="libcxx;libcxxabi;libunwind;compiler-rt" > > This is the same configuration the test-release.sh script uses, so I'm > hoping this will also fix all the test failures we've been seeing when > building the releases. > > Fixes #58680 This reverts commit 7f215b1.
Re-apply 7f215b1, which was reverted in a9e3d23. The orginal commit uncovered a bug that was fixed by 4701f77. Fixes llvm#58680
The test-depends target contained all the dependencies needed to run the runtimes tests, but it was never added as a dependency of check-all. This caused some of the tsan tests to fail, since the custom libcxx build the tests were looking for was never built. Besides the tsan failures, this fixes all the other test failures I was seeing with: cmake -G Ninja -B release-build -S llvm \ -DCMAKE_POSITION_INDEPENDENT_CODE=ON \ -DCMAKE_BUILD_TYPE=Release \ -DLLVM_ENABLE_ASSERTIONS=OFF \ -DLLVM_ENABLE_PROJECTS="clang;lld" \ -DLLVM_ENABLE_RUNTIMES="libcxx;libcxxabi;libunwind;compiler-rt" This is the same configuration the test-release.sh script uses, so I'm hoping this will also fix all the test failures we've been seeing when building the releases. Fixes llvm#58680 (cherry picked from commit 7f215b1)
Build for the full LLVM project (minus libc) completes, but fails tests due to a missing library file when running
check-all
. What could be causing this?Error message:
Build command:
The text was updated successfully, but these errors were encountered: