-
Notifications
You must be signed in to change notification settings - Fork 63
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
Dynamic linking to LLVM8 doesn't work #170
Comments
Apologies for the delay - finally had a chance to look at this today. To fix this we need to make LLVM part of the the index df4323d..41815e1 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -216,8 +216,9 @@ add_library(oclgrind ${CORE_LIB_TYPE}
target_link_libraries(oclgrind PRIVATE ${CORE_EXTRA_LIBS}
clangCodeGen clangFrontend clangSerialization clangDriver
clangParse clangSema clangAnalysis clangEdit clangAST clangASTMatchers
- clangLex clangBasic
- ${LLVM_LIBS})
+ clangLex clangBasic)
+
+target_link_libraries(oclgrind PUBLIC LLVM)
if ("${CMAKE_SYSTEM_NAME}" STREQUAL "Windows")
target_link_libraries(oclgrind PRIVATE Version)``` |
@jrprice I was also affected by #164 (comment) . Linux, Fedora 31. This change fixed it for me also:
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Following the workaround given in #164 I changed the CMakeLists.txt file on the latest master commit of Oclgrind to link to LLVM dynamically on Manjaro Linux (which is basically Arch). Unfortunately, it doesn't compile:
Is there any reasonable workaround for this, or is it a case of ABI incompatibility that needs to be addressed?
The text was updated successfully, but these errors were encountered: