-
Notifications
You must be signed in to change notification settings - Fork 3.9k
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
BCC does not support LLVLM 7 or earlier anymore with the master branch #3881
Comments
see #3808 |
confirmed on similar platform to ubuntu 20.04 with llvm7. |
Same error, different line:
Edit: got successful compile of |
ubuntu 18.04 and llvm 10, compile fail when checkout v0.24.0 from master , delete repo, and reclone the bcc based on v0.24.0 , got successful compile ( maybe the libbpf submodule affect |
I installed llvm-11 in Ubuntu 18.04 using llvm.sh script provided in https://apt.llvm.org/ . I was able to compile the latest bcc master successfully. I had to manually update CMakeLists to point to the new installation but the rest worked fine: To install llvm-11:
CMakeLists diff:
|
By trying to compile BCC on Ubuntu (both 18.04 and 20.04, GCC 7.5+llvm6 and GCC 9.3+llvm7, respectively), it fails as per the log:
Checking the autogenerated doxygen, I see indeed the function
getName()
does not have an argument anymore, but the headers for both llvm 7 and llvm 6 contain onlystd::error_code SectionRef::getName(StringRef &Result) const
This was changed in this commit of the llvm project (llvm/llvm-project@bcc00e1afbdc1). Looking at the releases history, I assume it was initially released in llvm9, as the commit dates to mid 2019.
BCC changed the usage of this
getName
function in the commit 8323d74 by @davemarchevsky, which was merged a couple of weeks ago to BCC's master branch.Should BCC require llvm9+ now? Not sure what solution should the project adopt.
INSTALL.md suggested llvm6 for Ubuntu 18.04 and llvm7 for Ubuntu 19 or 20.
The text was updated successfully, but these errors were encountered: