Skip to content
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

error: namespace "Eigen::half_impl" has no member "__half_raw" when building latest TensorFlow/CUDA 9.0 #15108

Closed
yaroslavvb opened this issue Dec 4, 2017 · 19 comments
Assignees

Comments

@yaroslavvb
Copy link
Contributor

When trying to build latest TensorFlow for CUDA 9.0/CuDNN 7.0, from today's head

Complete instructions to reproduce are here, but basically I followed steps in in Dockerfile.devel-gpu to install dependencies, and then did bazel build -c opt --config=cuda

Here are the errors

INFO: From Compiling tensorflow/stream_executor/cuda/cuda_blas.cc:
tensorflow/stream_executor/cuda/cuda_blas.cc: In function 'cudaDataType_t perftools::gputools::cuda::{anonymous}::CUDAComputationType(perftools::gputools::blas::ComputationType)':
tensorflow/stream_executor/cuda/cuda_blas.cc:527:1: warning: control reaches end of non-void function [-Wreturn-type]
 }
 ^
INFO: From Compiling tensorflow/core/kernels/dense_to_sparse_batch_dataset_op.cc:
tensorflow/core/kernels/dense_to_sparse_batch_dataset_op.cc: In member function 'virtual void tensorflow::{anonymous}::DenseToSparseBatchDatasetOp::MakeDataset(tensorflow::OpKernelContext*, tensorflow::DatasetBase*, tensorflow::DatasetBase**)':
tensorflow/core/kernels/dense_to_sparse_batch_dataset_op.cc:83:71: warning: 'batch_size' may be used uninitialized in this function [-Wmaybe-uninitialized]
         : batch_size_(batch_size), row_shape_(row_shape), input_(input) {
                                                                       ^
tensorflow/core/kernels/dense_to_sparse_batch_dataset_op.cc:41:11: note: 'batch_size' was declared here
     int64 batch_size;
           ^
INFO: From Compiling tensorflow/core/kernels/histogram_op_gpu.cu.cc:
./tensorflow/core/util/cuda_kernel_helper.h(109): error: namespace "Eigen::half_impl" has no member "__half_raw"

./tensorflow/core/util/cuda_kernel_helper.h(109): error: expected a ")"

./tensorflow/core/util/cuda_kernel_helper.h(110): error: namespace "Eigen::half_impl" has no member "__half_raw"

./tensorflow/core/util/cuda_kernel_helper.h(113): error: namespace "Eigen::half_impl" has no member "__half_raw"

./tensorflow/core/util/cuda_kernel_helper.h(113): error: expected a ";"

./tensorflow/core/util/cuda_kernel_helper.h(119): error: namespace "Eigen::half_impl" has no member "__half_raw"

./tensorflow/core/util/cuda_kernel_helper.h(119): error: expected a ")"

./tensorflow/core/util/cuda_kernel_helper.h(120): error: namespace "Eigen::half_impl" has no member "__half_raw"

./tensorflow/core/util/cuda_kernel_helper.h(123): error: namespace "Eigen::half_impl" has no member "__half_raw"

./tensorflow/core/util/cuda_kernel_helper.h(123): error: expected a ";"

10 errors detected in the compilation of "/tmp/tmpxft_000137d9_00000000-6_histog

@gunan
Copy link
Contributor

gunan commented Dec 4, 2017

#14770 should fix this issue.

@yzhang092
Copy link

#14770 seems not fix this issue. Still get error from cuda_kernel_helper

@nevakanezzar
Copy link

seconded, same issue.

@maxiaodong97
Copy link

looking into the history the breaking code introduced yesterday by
Wrappers for CUDA 9 warp-synchronous intrinsics.

@yaroslavvb
Copy link
Contributor Author

Cc @zheng-xq

@gunan
Copy link
Contributor

gunan commented Dec 7, 2017

Looks like I was wrong, the PR I mentioned did not fix this completely. However this is fixed internally.
@caisq when are we planning the next push from internal to github?

@caisq
Copy link
Contributor

caisq commented Dec 7, 2017

I'll push as soon as the ongoing pull is complete, @gunan

@JackShen2015
Copy link

I tried to build it, but it seems that the issue still exist

@castleguarders
Copy link

FYI, Issue still there in case the push already happened.

@caisq
Copy link
Contributor

caisq commented Dec 7, 2017

The push is open in PR #15174, awaiting approval.

@gunan
Copy link
Contributor

gunan commented Dec 14, 2017

This should now be fixed.

@gunan gunan closed this as completed Dec 14, 2017
@samikama
Copy link
Contributor

This is back again. Commit 1ad680d seems to break it again.

@David-Levinthal
Copy link

just leaving some of the build error messages from a cuda9.1 build with openmpi
INFO: From Compiling tensorflow/core/kernels/gather_functor_gpu.cu.cc:^M
./tensorflow/core/util/cuda_kernel_helper.h(116): error: namespace "Eigen::half_impl" has no member "__half_raw"

./tensorflow/core/util/cuda_kernel_helper.h(116): error: expected a ")"

./tensorflow/core/util/cuda_kernel_helper.h(117): error: namespace "Eigen::half_impl" has no member "__half_raw"

./tensorflow/core/util/cuda_kernel_helper.h(120): error: namespace "Eigen::half_impl" has no member "__half_raw"

./tensorflow/core/util/cuda_kernel_helper.h(120): error: expected a ";"

./tensorflow/core/util/cuda_kernel_helper.h(126): error: namespace "Eigen::half_impl" has no member "__half_raw"

./tensorflow/core/util/cuda_kernel_helper.h(126): error: expected a ")"

./tensorflow/core/util/cuda_kernel_helper.h(127): error: namespace "Eigen::half_impl" has no member "__half_raw"

./tensorflow/core/util/cuda_kernel_helper.h(130): error: namespace "Eigen::half_impl" has no member "__half_raw"

./tensorflow/core/util/cuda_kernel_helper.h(130): error: expected a ";"

10 errors detected in the compilation of "/tmp/tmpxft_00004514_00000000-6_gather_functor_gpu.cu.cpp1.ii".
ERROR: /home/levinth/tensorflow/tensorflow/core/kernels/BUILD:1139:1: output 'tensorflow/core/kernels/_objs/gather_functor_gpu/tensorflow/core/kernels/gather_functor_gpu.cu.pic.o' was not created

@gunan
Copy link
Contributor

gunan commented Dec 22, 2017

@reedwm @zhengxq FYI.

I think we have an internal fix, we are waiting for that to be pushed out.
@reedwm, could you confirm?

@reedwm
Copy link
Member

reedwm commented Dec 22, 2017

1ad680d has been rolled back internally, so this should be fixed internally. Thank you @samikama for identifying the specific commit that broke it.

@David-Levinthal
Copy link

when will we see the fix externally?

@gunan
Copy link
Contributor

gunan commented Dec 23, 2017 via email

@David-Levinthal
Copy link

David-Levinthal commented Dec 23, 2017 via email

@gunan
Copy link
Contributor

gunan commented Dec 23, 2017

Thanks for verifying!
Then I am closing this issue.

@gunan gunan closed this as completed Dec 23, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests