-
Notifications
You must be signed in to change notification settings - Fork 10.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
build : pass all warning flags to nvcc via -Xcompiler #5570
Conversation
Looks like PR #3952 broke the Makefile. I promise, this stuff all worked when I wrote it :) |
We intentionally craft a GF_CC for CUDA that will fail but print the desired version anyway. So instead of checking whether the version command was successful, we need to check if the command printed anything.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This seems to fix it.
For the record this made my builds stop working with this error:
Reverting this change made it go through, though I imagine it's more likely that something else is causing a true error and your change is making it be accurately reported? |
I noticed that with |
|
* build : pass all warning flags to nvcc via -Xcompiler * make : fix apparent mis-merge from ggml-org#3952 * make : fix incorrect GF_CC_VER for CUDA host compiler
* build : pass all warning flags to nvcc via -Xcompiler * make : fix apparent mis-merge from ggml-org#3952 * make : fix incorrect GF_CC_VER for CUDA host compiler
Ref #5543
Fixes #5542