<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1">
<style type="text/css" style="display:none;"> P {margin-top:0;margin-bottom:0;} </style>
</head>
<body dir="ltr">
<div>This commit</div>
<div><br>
</div>
<a href="https://github.com/xiph/opus/commit/94b68f341cadd5433a10d346c1c248a641d8be57">https://github.com/xiph/opus/commit/94b68f341cadd5433a10d346c1c248a641d8be57</a>
<div><br>
</div>
<div>Enabled HAVE_LRINTF defined in CMake builds.</div>
<div><br>
</div>
<div>As later versions of visual studio have LRINTF it got enabled by default due to precedence over SSE in MSVC. The use of lrintf is a lot slower which can easily be seen in the tests</div>
<div><br>
</div>
<div>From test result Windows X64 (similar results on X86): </div>
<div><span style="color: rgb(0, 0, 0); font-family: SFMono-Regular, Consolas, "Liberation Mono", Menlo, monospace; font-size: 12px; background-color: rgb(255, 255, 255); display: inline !important;">LRINTF 4/4 Test #4: test_opus_encode ................. Passed
225.96 sec</span></div>
<div><span style="color: rgb(0, 0, 0); font-family: SFMono-Regular, Consolas, "Liberation Mono", Menlo, monospace; font-size: 12px; background-color: rgb(255, 255, 255); display: inline !important;">SSE </span><span style="color: rgb(246, 248, 250); font-family: SFMono-Regular, Consolas, "Liberation Mono", Menlo, monospace; font-size: 12px; background-color: rgb(47, 54, 61); display: inline !important"><span style="font-family: SFMono-Regular, Consolas, "Liberation Mono", Menlo, monospace; background-color: rgb(255, 255, 255); display: inline !important; color: rgb(0, 0, 0);">4/4
Test #4: test_opus_encode ................. Passed 56.30 sec</span></span></div>
<div><br>
</div>
<div>The patch simply moves MSVC SSE and ASM up before HAVE_LRINT. </div>
<div>Change the types to opus types and verified that mm_load compiled to same</div>
<div>code as mm_set in later versions of MSVC x64.</div>
<div><br>
</div>
<div>//Marcus</div>
<div><span style="color: rgb(246, 248, 250); font-family: SFMono-Regular, Consolas, "Liberation Mono", Menlo, monospace; font-size: 12px; background-color: rgb(47, 54, 61); display: inline !important"><span style="font-family: SFMono-Regular, Consolas, "Liberation Mono", Menlo, monospace; background-color: rgb(36, 41, 46); display: inline !important"><br>
</span></span></div>
<div><span style="color: rgb(246, 248, 250); font-family: SFMono-Regular, Consolas, "Liberation Mono", Menlo, monospace; font-size: 12px; background-color: rgb(47, 54, 61); display: inline !important"><span style="font-family: SFMono-Regular, Consolas, "Liberation Mono", Menlo, monospace; background-color: rgb(36, 41, 46); display: inline !important"><br>
</span></span></div>
</body>
</html>