<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Apr 14, 2020 at 5:42 PM Rémi Denis-Courmont <<a href="mailto:remi@remlab.net">remi@remlab.net</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">Le tiistaina 14. huhtikuuta 2020, 18.14.52 EEST Thomas Guillem a écrit :<br>
> On Tue, Apr 14, 2020, at 16:57, Alexandre Janniaux wrote:<br>
> > Hi,<br>
> > <br>
> > On Tue, Apr 14, 2020 at 05:45:18PM +0300, Rémi Denis-Courmont wrote:<br>
> > > Le tiistaina 14. huhtikuuta 2020, 17.35.42 EEST Alexandre Janniaux a <br>
écrit :<br>
> > > > Hi,<br>
> > > > <br>
> > > > Isn't vlc_CPU checked only in initialization steps to choose<br>
> > > > the correct variant?<br>
> > > <br>
> > > No.<br>
> > <br>
> > Can't we just remove the few case where it's not calling it<br>
> > during initialization? At the same time, it would remove<br>
> > uneeded atomic operation on hot paths.<br>
> <br>
> Indeed, we could fix the few case where vlc_CPU is checked often.<br>
> <br>
> In that case, I'm OK with this patch, that is simplifying it a lot.<br>
<br>
And I'm not OK with this complete nonsense patch.<br>
<br>
If this file is too complicated for you, nobody's asking you to touch it. Just <br>
don't remove other people's optimisations.<br></blockquote><div><br></div><div>Can you explain how this removes any optimization?</div><div>It just force check for the feature to have been detected on the first call to vlc_CPU.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
-- <br>
Rémi Denis-Courmont<br>
Hagalund ny stad, Nylands republik<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></blockquote></div></div>