[opus] Misc patches
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
Is this an extra feature the appveyor environment doesn't apply, or is
the version guard incorrect?
On Mon, 2020-06-08 at 05:39 +0000, Marcus Asteborg wrote:
> 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
> 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.
> opus mailing list
> opus at xiph.org
More information about the opus