<div>Hi,</div><div><br></div><div>any objections to replace all __m128 initialization with _mm_set_ps?</div><div>E.g.:</div><div>-    const __m128 half  = {0.5,0.5,0.5,0.5};</div><div>+    const __m128 half  = _mm_set_ps(0.5,0.5,0.5,0.5);</div>

<div><br></div><div>This makes it compile on PGI. (I also think the initialization is otherwise not standard conform).</div><div><br></div><div>But do we actually want to support all compilers (i.e. PGI)? Since compiler bugs are more common than one would wish, I&#39;m not sure it is a good situation that users are using a compilers which haven&#39;t been tested/used by any developer. Since the test-suite is currently suboptimal the administrator installing GROMACS has no way to check whether the installation is OK. I think this is important because many supercomputers here in the US have PGI as the default compiler. Most I&#39;ve checked have GROMACS installed (still 4.0.7) and usually compiled with PGI.</div>

<div><br></div><div>Thus I suggest that we add a warning to configure/cmake if the compiler is not gcc, icc, or xlc. I had before problems with PGI (don&#39;t remember version) and I know GROMACS doesn&#39;t compile currently with Pathscale or Open64 (both internal compiler errors). Independent of this I suggest to change the initialization to _mm_set_ps to be standard conforming. </div>

<div><br></div><div>Roland</div><br>-- <br>ORNL/UT Center for Molecular Biophysics <a href="http://cmb.ornl.gov">cmb.ornl.gov</a><br>865-241-1537, ORNL PO BOX 2008 MS6309<br>