[eigen] RE: minor warning fix

[ Thread Index | Date Index | More lists.tuxfamily.org/eigen Archives ]

Sorry, but I also forgot one other warning, which is here:


      IsAligned = bool(EIGEN_MAX_ALIGN_BYTES>0) & !(Options_&DontAlign),


In unsupported/Eigen/CXX11/src/Tensor/Tensor.h


The warning is about potential confusion over the operator precedence with the & (don’t ask me why this is somehow confusing). Changing it to this removes the warning:


      IsAligned = (bool(EIGEN_MAX_ALIGN_BYTES>0)) & !(Options_&DontAlign),


In both cases, I’m compiling with MSVC 2015.




From: Matt Richardson [mailto:mattri@xxxxxxxxxxxxx]
Sent: Friday, July 21, 2017 1:52 PM
To: eigen@xxxxxxxxxxxxxxxxxxx
Subject: [eigen] minor warning fix


This sender failed our fraud detection checks and may not be who they appear to be. Learn about spoofing




This function in unsupported\Eigen\CXX11\src\Tensor\TensorDeviceCuda.h (lines 290-292):


EIGEN_STRONG_INLINE int sharedMemPerBlock() const {

  return stream_->deviceProperties().sharedMemPerBlock;



Is causing a compile warning (about the conversion from size_t to int) for me. I don’t have access to Bugzilla to report the bug. Is it possible to change the return type to size_t?







Mail converted by MHonArc 2.6.19+ http://listengine.tuxfamily.org/