diff options
author | Eugene Zhulenev <ezhulenev@google.com> | 2018-07-19 17:37:03 -0700 |
---|---|---|
committer | Eugene Zhulenev <ezhulenev@google.com> | 2018-07-19 17:37:03 -0700 |
commit | c58b874727717c88f5026d29e3eadc9d1b19653d (patch) | |
tree | c3d194cdc9582dbac6b2ef3b8fb3ab22d163d805 /test/sparse_product.cpp | |
parent | 2424e3b7aca514783fea4026775068827b8ee915 (diff) |
PR430: Convert count to the reducer type in MeanReducer
Without explicit conversion Tensorflow fails to compile, pset1 template deduction fails.
cannot convert '((const Eigen::internal::MeanReducer<Eigen::half>*)this)
->Eigen::internal::MeanReducer<Eigen::half>::packetCount_'
(type 'const DenseIndex {aka const long int}')
to type 'const type& {aka const Eigen::half&}'
return pdiv(vaccum, pset1<Packet>(packetCount_));
Honestly I’m not sure why it works in Eigen tests, because Eigen::half constructor is explicit, and why it stopped working in TF, I didn’t find any relevant changes since previous Eigen upgrade.
static_cast<T>(packetCount_) - breaks cxx11_tensor_reductions test for Eigen::half, also quite surprising.
Diffstat (limited to 'test/sparse_product.cpp')
0 files changed, 0 insertions, 0 deletions