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

Replace math_functions.h with cuda_runtime.h to remove CUDA compile w… #6681

Open
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
2 participants
@chenzeyuczy
Copy link
Contributor

chenzeyuczy commented Jan 25, 2019

When compile with CUDA10, there's a warning like this.
In file included from src/caffe/util/math_functions.cu:1:0:
/usr/local/cuda/include/math_functions.h:54:2: warning: #warning "math_functions.h is an internal header file and must not be used directly. This file will be removed in a future CUDA release. Please use cuda_runtime_api.h or cuda_runtime.h instead." [-Wcpp]
#warning "math_functions.h is an internal header file and must not be used directly. This file will be removed in a future CUDA release. Please use cuda_runtime_api.h or cuda_runtime.h instead."
This PR aims to remove the deprecated header file.

@@ -1,4 +1,4 @@
#include <math_functions.h> // CUDA's, not caffe's, for fabs, signbit
#include <cuda_runtime.h> // CUDA's, not caffe's, for fabs, signbit

This comment has been minimized.

@shahzadlone

shahzadlone Feb 5, 2019

What happened to math_functions.h

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment