-
Notifications
You must be signed in to change notification settings - Fork 161
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
Failure to build 1.0.8744 : IGC/VectorCompiler/CMCL/tools/Translator/Main.cpp:46: undefined reference to `cmcl::translateBuiltins(llvm::Module&)' unless '-z,undefs' specified as LD flag #212
Comments
The need for Also you might want:
but see #210 for LLD. |
@ArchangeGabriel thanks for the reply. I've:
However, it doesn't seem to help, I am still stuck with:
or
build.log from llvm/lld: I've created and I am using copr repository for building - which contains updated spirv llvm and packaged clang-opencl (which is still missing in Fedora and which I'll include once I get the entire chain up n working): https://copr.fedorainfracloud.org/coprs/frantisekz/intel-media-driver-free/ |
@JacekDanecki won't you have any idea what can be the cause of this? I've seen you've done a bit of Fedora/Copr packaging of igc and others... Thanks! |
The issue goes away by changing LDFLAGS from: to: '-Wl,-z,relro -Wl,-z,undefs,-z,now -flto' I'll leave the ticket open with the title changed, however, feel free to close it if this is expected. |
The issue was in Fedora defaulting to -DBUILD_SHARED_LIBS:BOOL=ON . |
Hi,
I am experimenting a bit with packaging bunch of Intel packages into Fedora (with ultimate goal of getting the media-driver there, including the kernels compilation). I need to use system llvm (13) for this instead of bundled llvm.
I am currently stuck with:
The entire buildlog could be viewed here:
build-igc.log
I've had to add bunch of other fixes (namely some -Wno-error and a fix around clang-opencl soname not being parsed correctly). I'll create issues for these later if that's desired.
I can upload the entire src.rpm set if that's necessary.
Thanks in advance!
The text was updated successfully, but these errors were encountered: