<html><head><meta http-equiv="Content-Type" content="text/html charset=windows-1252"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;">I’ll create a patch tonight, unless somebody else beats me to it :-)<div><br></div><div>Cheers,</div><div><br></div><div>Erik</div><div><br><div><div>On 15 Mar 2014, at 13:39, David van der Spoel <<a href="mailto:spoel@xray.bmc.uu.se">spoel@xray.bmc.uu.se</a>> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div style="font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;">On 2014-03-15 12:49, Erik Lindahl wrote:<br><blockquote type="cite">It might be the sizeof() call that screws things up by returning a 64-bit value, in combination with the compiler not understanding that it is a constant.<br><br>Could you cry changing those sizeof() to "4" and see if that works?<br><br></blockquote>Yes, that works!<br><br><blockquote type="cite">Cheers,<br><br>Erik<br><br>--<br>Erik Lindahl <<a href="mailto:erik.lindahl@scilifelab.se">erik.lindahl@scilifelab.se</a>><br>Professor of Biophysics<br>Science for Life Laboratory<br>Stockholm University & KTH<br>Office (SciLifeLab): +46 8 524 81567<br>Cell (Sweden): +46 73 4618050<br>Cell (US): 1 267 307 8746<br><br><br><blockquote type="cite">On Mar 15, 2014, at 12:19 PM, David van der Spoel <<a href="mailto:spoel@xray.bmc.uu.se">spoel@xray.bmc.uu.se</a>> wrote:<br><br><blockquote type="cite">On 2014-03-15 11:44, David van der Spoel wrote:<br>In file included from<br>/home/spoel/GG/master/gromacs/src/gromacs/simd/simd.h:126:0,<br> from<br>/home/spoel/GG/master/gromacs/src/gromacs/simd/tests/simd.h:79,<br> from<br>/home/spoel/GG/master/gromacs/src/gromacs/simd/tests/simd_integer.cpp:39:<br>/home/spoel/GG/master/gromacs/src/gromacs/simd/tests/simd_integer.cpp:<br>In member function 'virtual void<br>gmx::test::{anonymous}::SimdIntegerTest_gmxSimdExtractI_Test::TestBody()':<br>/home/spoel/GG/master/gromacs/src/gromacs/simd/impl_x86_sse2/impl_x86_sse2.h:123:92:<br>error: the last argument must be an 8-bit immediate<br> #define gmx_simd_extract_fi(x, i)<br>_mm_cvtsi128_si32(_mm_srli_si128((x), sizeof(int) * (i)))<br><br> ^<br>/home/spoel/GG/master/gromacs/src/gromacs/simd/simd.h:665:46: note: in<br>expansion of macro 'gmx_simd_extract_fi'<br> # define gmx_simd_extract_i gmx_simd_extract_fi<br> ^<br>/home/spoel/GG/master/gromacs/src/gromacs/simd/tests/simd_integer.cpp:154:25:<br>note: in expansion of macro 'gmx_simd_extract_i'<br> extracted_int = gmx_simd_extract_i(simd, 1);<br> ^<br>/home/spoel/GG/master/gromacs/src/gromacs/simd/impl_x86_sse2/impl_x86_sse2.h:123:92:<br>error: the last argument must be an 8-bit immediate<br> #define gmx_simd_extract_fi(x, i)<br>_mm_cvtsi128_si32(_mm_srli_si128((x), sizeof(int) * (i)))<br><br> ^<br>/home/spoel/GG/master/gromacs/src/gromacs/simd/simd.h:665:46: note: in<br>expansion of macro 'gmx_simd_extract_fi'<br> # define gmx_simd_extract_i gmx_simd_extract_fi<br> ^<br>/home/spoel/GG/master/gromacs/src/gromacs/simd/tests/simd_integer.cpp:159:25:<br>note: in expansion of macro 'gmx_simd_extract_i'<br> extracted_int = gmx_simd_extract_i(simd, 3);<br><br>Any clues?<br></blockquote>To follow up: it compiles fine with SSE4.1 but not with SSE2.<br><br><br>--<br>David van der Spoel, Ph.D., Professor of Biology<br>Dept. of Cell & Molec. Biol., Uppsala University.<br>Box 596, 75124 Uppsala, Sweden. Phone: +46184714205.<br><a href="mailto:spoel@xray.bmc.uu.se">spoel@xray.bmc.uu.se</a> <a href="http://folding.bmc.uu.se">http://folding.bmc.uu.se</a><br>--<br>Gromacs Developers mailing list<br><br>* Please search the archive at <a href="http://www.gromacs.org/Support/Mailing_Lists/GMX-developers_List">http://www.gromacs.org/Support/Mailing_Lists/GMX-developers_List</a> before posting!<br><br>* Can't post? Read <a href="http://www.gromacs.org/Support/Mailing_Lists">http://www.gromacs.org/Support/Mailing_Lists</a><br><br>* For (un)subscribe requests visit<br><a href="https://maillist.sys.kth.se/mailman/listinfo/gromacs.org_gmx-developers">https://maillist.sys.kth.se/mailman/listinfo/gromacs.org_gmx-developers</a> or send a mail to <a href="mailto:gmx-developers-request@gromacs.org">gmx-developers-request@gromacs.org</a>.<br></blockquote></blockquote><br><br>--<span class="Apple-converted-space"> </span><br>David van der Spoel, Ph.D., Professor of Biology<br>Dept. of Cell & Molec. Biol., Uppsala University.<br>Box 596, 75124 Uppsala, Sweden. Phone:<span class="Apple-tab-span" style="white-space: pre;"> </span>+46184714205.<br><a href="mailto:spoel@xray.bmc.uu.se">spoel@xray.bmc.uu.se</a><span class="Apple-converted-space"> </span> <a href="http://folding.bmc.uu.se/">http://folding.bmc.uu.se</a><br>--<span class="Apple-converted-space"> </span><br>Gromacs Developers mailing list<br><br>* Please search the archive at<span class="Apple-converted-space"> </span><a href="http://www.gromacs.org/Support/Mailing_Lists/GMX-developers_List">http://www.gromacs.org/Support/Mailing_Lists/GMX-developers_List</a><span class="Apple-converted-space"> </span>before posting!<br><br>* Can't post? Read<span class="Apple-converted-space"> </span><a href="http://www.gromacs.org/Support/Mailing_Lists">http://www.gromacs.org/Support/Mailing_Lists</a><br><br>* For (un)subscribe requests visit<br><a href="https://maillist.sys.kth.se/mailman/listinfo/gromacs.org_gmx-developers">https://maillist.sys.kth.se/mailman/listinfo/gromacs.org_gmx-developers</a><span class="Apple-converted-space"> </span>or send a mail to<span class="Apple-converted-space"> </span><a href="mailto:gmx-developers-request@gromacs.org">gmx-developers-request@gromacs.org</a>.</div></blockquote></div><br></div></body></html>