diff options
author | Gael Guennebaud <g.gael@free.fr> | 2018-10-08 12:01:18 +0200 |
---|---|---|
committer | Gael Guennebaud <g.gael@free.fr> | 2018-10-08 12:01:18 +0200 |
commit | 64b1a153182ec4ce9976291c29db2099b30e77d8 (patch) | |
tree | dd514b08cc08c9769cf2f20747741f39b02df1cd /unsupported/Eigen/CXX11/src/Tensor/TensorScan.h | |
parent | c9643f4a6f6827908f4d351bb8a3f7a0027b3769 (diff) |
Workaround stupid warning
Diffstat (limited to 'unsupported/Eigen/CXX11/src/Tensor/TensorScan.h')
-rw-r--r-- | unsupported/Eigen/CXX11/src/Tensor/TensorScan.h | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/unsupported/Eigen/CXX11/src/Tensor/TensorScan.h b/unsupported/Eigen/CXX11/src/Tensor/TensorScan.h index 641366d9d..64f10d0a4 100644 --- a/unsupported/Eigen/CXX11/src/Tensor/TensorScan.h +++ b/unsupported/Eigen/CXX11/src/Tensor/TensorScan.h @@ -124,7 +124,11 @@ struct TensorEvaluator<const TensorScanOp<Op, ArgType>, Device> { m_stride = m_stride * dims[i]; } } else { - for (int i = NumDims - 1; i > op.axis(); --i) { + // dims can only be indexed through unsigned integers, + // so let's use an unsigned type to let the compiler knows. + // This prevents stupid warnings: ""'*((void*)(& evaluator)+64)[18446744073709551615]' may be used uninitialized in this function" + unsigned int axis = internal::convert_index<unsigned int>(op.axis()); + for (unsigned int i = NumDims - 1; i > axis; --i) { m_stride = m_stride * dims[i]; } } |