[opus] Misc patches

Ralph Giles giles at thaumas.net
Fri Jun 12 02:31:11 UTC 2020


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.com/project/rillian/opus/builds/33474422

Is this an extra feature the appveyor environment doesn't apply, or is
the version guard incorrect?

 -r

On Mon, 2020-06-08 at 05:39 +0000, Marcus Asteborg wrote:
> Hi,
> 
> Here are 3 suggested patches.
> Build test for cmake and run the test in gitlab-ci.
> Disable the message box on Windows on abort that cause test hangs in
> CI.
> 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
> _______________________________________________
> opus mailing list
> opus at xiph.org
> http://lists.xiph.org/mailman/listinfo/opus



More information about the opus mailing list