Re: [eigen] clang analyzer warnings

[Rick Mann <rmann@xxxxxxxxxxxxxxx>]

Thanks!

On Oct 26, 2013, at 05:00 , Gael Guennebaud <gael.guennebaud@xxxxxxxxx> wrote:

> thank you for the report. it was easier to fix them while checking them:
> 
> https://bitbucket.org/eigen/eigen/commits/2ffba8f5dde4/
> Changeset:   2ffba8f5dde4
> User:        ggael
> Date:        2013-10-26 13:59:02
> Summary:     fix a few "dead stores" warnings
> 
> 
> On Sat, Oct 26, 2013 at 11:33 AM, Thomas Capricelli <orzel@xxxxxxxxxxxxxxx> wrote:
> Il 26/10/2013 02:57, Rick Mann ha scritto:
> > Hi guys. I just ran the clang static analyzer on Eigen 3.2.0, and there are only a few (trivial) complaints. There are four "dead stores" (assignments to variables never referenced). MathFunction.h lines 129 ("tmp"), 198 ("tmp"), 282, ("xmm2"), and 381 ("xmm2”).
> 
> (that's src/Core/arch/SSE/MathFunctions.h)
> 
> 
> 
> --
> Thomas Capricelli <orzel@xxxxxxxxxxxxxxx>
> http://www.freehackers.org/thomas/
> 
> 
> 


-- 
Rick

Attachment: signature.asc
Description: Message signed with OpenPGP using GPGMail