<div dir="ltr">Hi,<div><br></div><div>Doesn't that imply I'd have to write a version per feature I want to test?</div><div><br></div><div>Like NO_OPTI for C, SSE2_OPTI, SSSE3 OPTI, SSE4_OPTI (if we happen to have dedicated code for this at some point), AVX2_OPTI, AVX512_OPTI (in the future).</div><div>I mean it can be done, in a .h file that we include everywhere we want to test, or just in vlc_common.h or smth.</div><div><br></div><div>But what is wrong with extending the API?</div><div>I get that it's not gonna be used outside of tool purposes, that would be odd, but it does hurt right? Does it?</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Apr 14, 2020 at 4:34 PM Thomas Guillem <<a href="mailto:thomas@gllm.fr">thomas@gllm.fr</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">FYI, this is what I did in copy.c to test with optims disabled:<br>
<br>
#ifdef COPY_TEST_NOOPTIM<br>
# undef vlc_CPU_SSE4_1<br>
# define vlc_CPU_SSE4_1() (0)<br>
# undef vlc_CPU_SSE3<br>
# define vlc_CPU_SSE3() (0)<br>
# undef vlc_CPU_SSSE3<br>
# define vlc_CPU_SSSE3() (0)<br>
# undef vlc_CPU_SSE2<br>
# define vlc_CPU_SSE2() (0)<br>
#endif<br>
<br>
On Tue, Apr 14, 2020, at 16:01, Rémi Denis-Courmont wrote:<br>
> Le tiistaina 14. huhtikuuta 2020, 13.40.15 EEST Victorien Le Couviour--Tuffet a <br>
> écrit :<br>
> > This is allows to temporarily mask a CPU feature.<br>
> > It can for example be useful for testing a function with several<br>
> > underlying SIMD implementations.<br>
> > <br>
> > It adds vlc_CPU_mask and vlc_CPU_unmask to the core API.<br>
> <br>
> This is neither thread-safe nor compatible with the pure/constant attributes, <br>
> which really should be set on vlc_CPU() (In fact, I thought they already were, <br>
> though it seems they are not).<br>
> <br>
> -- <br>
> Rémi Denis-Courmont<br>
> Tapiolan uusi kaupunki, Uudenmaan tasavalta<br>
> <br>
> <br>
> <br>
> _______________________________________________<br>
> vlc-devel mailing list<br>
> To unsubscribe or modify your subscription options:<br>
> <a href="https://mailman.videolan.org/listinfo/vlc-devel" rel="noreferrer" target="_blank">https://mailman.videolan.org/listinfo/vlc-devel</a><br>
_______________________________________________<br>
vlc-devel mailing list<br>
To unsubscribe or modify your subscription options:<br>
<a href="https://mailman.videolan.org/listinfo/vlc-devel" rel="noreferrer" target="_blank">https://mailman.videolan.org/listinfo/vlc-devel</a></blockquote></div>