<br><br><div class="gmail_quote">On Wed, Aug 24, 2011 at 2:16 PM, Ken Nielson <span dir="ltr">&lt;<a href="mailto:knielson@adaptivecomputing.com">knielson@adaptivecomputing.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div class="im">----- Original Message -----<br>
&gt; From: &quot;Glen Beane&quot; &lt;<a href="mailto:glen.beane@gmail.com">glen.beane@gmail.com</a>&gt;<br>
&gt; To: &quot;Torque Developers mailing list&quot; &lt;<a href="mailto:torquedev@supercluster.org">torquedev@supercluster.org</a>&gt;<br>
&gt; Sent: Wednesday, August 24, 2011 12:07:59 PM<br>
&gt; Subject: Re: [torquedev] Release Candidate for TORQUE 2.5.8<br>
</div><div><div></div><div class="h5">&gt; On Wed, Aug 24, 2011 at 1:54 PM, Ken Nielson &lt;<br>
&gt; <a href="mailto:knielson@adaptivecomputing.com">knielson@adaptivecomputing.com</a> &gt; wrote:<br>
&gt;<br>
&gt;<br>
&gt; There is a new release candidate for TORQUE available. This has the<br>
&gt; fixes for the compiler warnings with the exception of<br>
&gt; process_request.c, function get_creds line 288. dereferencing<br>
&gt; type-punned pointer will break strict-aliasing rules.<br>
&gt;<br>
&gt; in my opinion, this is a warning that shouldn&#39;t be ignored<br>
&gt; indefinitely (not that anyone has suggested it). I think it would be<br>
&gt; good to fix before the official release. I would also add<br>
&gt; -fno-strict-aliasing to my CFLAGS until it were fixed.<br>
&gt;<br>
&gt; I would volunteer to the look at the code responsible for the warning,<br>
&gt; but I can&#39;t guarantee I would have a chance any time soon.<br>
&gt;<br>
</div></div><div class="im">&gt; _______________________________________________<br>
&gt; torquedev mailing list<br>
&gt; <a href="mailto:torquedev@supercluster.org">torquedev@supercluster.org</a><br>
&gt; <a href="http://www.supercluster.org/mailman/listinfo/torquedev" target="_blank">http://www.supercluster.org/mailman/listinfo/torquedev</a><br>
<br>
</div>Glen,<br>
<br>
Thanks for volunteering. I agree with you that we should fix this but I did not want to postpone a release candidate for a warning. This particular warning only makes it so certain optimizations cannot be applied to the code. I started to look at a solution and did not see a quick fix. Like you I am busy with other priorities.<br>

<br>
If you get time to fix this please do.<br>
<br>
For now I propose that we do not postpone the release of 2.5.8 based on this warning.<br></blockquote></div><br>I didn&#39;t volunteer :)   What I meant was that I would except that I don&#39;t think I would get to it any time soon, so I won&#39;t commit to anything right now.<br>
<br>actually, the warning doesn&#39;t mean that certain optimizations wont&#39; be done.  It means the compiler can generate incorrect code in this case.  The warning should either be fixed, or -fno-strict-aliasing should be added to the CFLAGS.  I looked at the code for about 5 minutes and it was not obvious to me what was going on.  punting is allowed with strict aliasing optimizations enabled if done with through an union though<br>
<br>