search for: intrin0

Sorry about that, let me check the correct version for the intrin0.h include guard. //Marcus ________________________________ From: Ralph Giles <giles at thaumas.net> Sent: Thursday, June 11, 2020 19:31 To: Marcus Asteborg <xnorpx at outlook.com>; opus at xiph.org <opus at xiph.org> Subject: Re: [opus] Misc patches Speaking of needing more comp...

...re 3 suggested patches. 1. Build test for cmake and run the test in gitlab-ci. 2. Disable the message box on Windows on abort that cause test hangs in CI. 3. Build time improvement by removing unnecessary includes of stdio.h in production code and change to a lighter header intrin.h -> intrin0.h (windows only). Attached screenshot of measurement but it resulted in 14% buildspeed compared to master on Windows 64 debug builds on my setup. As most improvements comes from change intrin.h the impact on other platforms will not be that much. //Marcus -------------- next part -------------- An...

Verified that intrin0.h was available in 1910, by building it locally on VS2017 version 15.0 MSVC++ 14.1 _MSC_VER == 1910 (Visual Studio 2017 version 15.0) https://en.wikipedia.org/wiki/Microsoft_Visual_C%2B%2B //Marcus ________________________________ From: Marcus Asteborg <xnorpx at outlook.com> Sent: Thurs...

...and run the test in gitlab-ci. OK, merged. > 2. Disable the message box on Windows on abort that cause test hangs in CI. Merged too. > 3. Build time improvement by removing unnecessary includes of stdio.h > in production code and change to a lighter header intrin.h -> > intrin0.h (windows only). Attached screenshot of measurement but it > resulted in 14% buildspeed compared to master on Windows 64 debug > builds on my setup. As most improvements comes from change intrin.h > the impact on other platforms will not be that much. I'm having a bit of...

Speaking of needing more complete ci feedback, the intrin0.h patch broke the appveyor build: > Microsoft (R) Build Engine version 14.0.25420.1 > [...] > :\projects\opus\celt\ecintrin.h(53): fatal error C1083: Cannot open include file: 'intrin0.h': No such file or directory [C:\projects\opus\win32\VS2015\opus.vcxproj] https://ci.appveyor....

...t; 1. Build test for cmake and run the test in gitlab-ci. > 2. Disable the message box on Windows on abort that cause test hangs in CI. > 3. Build time improvement by removing unnecessary includes of stdio.h > in production code and change to a lighter header intrin.h -> > intrin0.h (windows only). Attached screenshot of measurement but it > resulted in 14% buildspeed compared to master on Windows 64 debug > builds on my setup. As most improvements comes from change intrin.h > the impact on other platforms will not be that much. I need to have a closer...

...and run the test in gitlab-ci. OK, merged. > 2. Disable the message box on Windows on abort that cause test hangs in CI. Merged too. > 3. Build time improvement by removing unnecessary includes of stdio.h > in production code and change to a lighter header intrin.h -> > intrin0.h (windows only). Attached screenshot of measurement but it > resulted in 14% buildspeed compared to master on Windows 64 debug > builds on my setup. As most improvements comes from change intrin.h > the impact on other platforms will not be that much. I'm having a bit of...