[eigen] RE: minor warning fix

[Matt Richardson <mattri@xxxxxxxxxxxxx>]

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.

 

Matt

 

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

Feedback

Hi,

 

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?

 

Thanks!

Matt