[opus] Misc patches

Marcus Asteborg xnorpx at outlook.com
Fri Jun 12 03:15:31 UTC 2020


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 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://nam10.safelinks.protection.outlook.com/?url=https%3A%2F%2Fci.appveyor.com%2Fproject%2Frillian%2Fopus%2Fbuilds%2F33474422&data=02%7C01%7C%7C4162e72343444c9c310608d80e78ad01%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C637275258756998152&sdata=1zMyaftA7NjAXL52YRlQtwyNG3Xaen2U7kngJ%2F8KZtU%3D&reserved=0

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
> https://nam10.safelinks.protection.outlook.com/?url=http%3A%2F%2Flists.xiph.org%2Fmailman%2Flistinfo%2Fopus&data=02%7C01%7C%7C4162e72343444c9c310608d80e78ad01%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C637275258757008148&sdata=wrR6%2FwZJoaV%2F7s36m3MglWx41ntOa3ZGXRwQh%2BU7cDU%3D&reserved=0

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.xiph.org/pipermail/opus/attachments/20200612/d563174e/attachment.html>


More information about the opus mailing list