<div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr">Hello,<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span style="color:rgb(85,85,85);letter-spacing:0.2px;white-space:nowrap">What problem are we solving?</span></blockquote><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">That we don't get fast paths in rounding.h by default on i686, I think?</blockquote><div><div>Yes, it adds optimization for rounding.h. On x86_64 msse is enabled by default (as example by gcc).</div><div>And additionally helps to fix autotools 32b build (missed linkage with m-lib if no optimization).</div></div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">This will make the code non-portable from an SSE capable system to a non-sse<br>capable system. That's a pretty old system at this point (Pentium III and Athlon<br>XP)</blockquote><div><div>As i found in gcc-docs (<a href="https://gcc.gnu.org/onlinedocs/gcc-4.5.3/gcc/i386-and-x86_002d64-Options.html">https://gcc.gnu.org/onlinedocs/gcc-4.5.3/gcc/i386-and-x86_002d64-Options.html</a>) exactly mentioned systems are supports SSE.</div><div>But in the patch i'm checking if -msse supported by compiler.</div><div>If its incorrect than does it mean that compiler doesn't based on system capabilities? And does it mean that adding option -msse4.1 is also enabled in wrong way?</div><div><br></div><div>I'm cloning gcc to check how it supports 'sse' and if it based on system capabilities. Also we may try to add some possibility to check cpu-capabilities (around cpuid).</div></div><div>But if we don't need such optimization, than it will be nice at least to fix autotools 32b build (<a href="https://patchwork.freedesktop.org/patch/258659/">https://patchwork.freedesktop.org/patch/258659/</a>) - i will update patch just by linking -lm with mesautil.</div><div><br></div><div>Please, advise.</div></div></div></div></div></div><br><div class="gmail_quote"><div dir="ltr">On Fri, Oct 26, 2018 at 9:54 PM Dylan Baker <<a href="mailto:dylan@pnwbakers.com">dylan@pnwbakers.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Quoting Matt Turner (2018-10-26 10:44:47)<br>
> On Fri, Oct 26, 2018 at 3:06 AM Sergii Romantsov<br>
> <<a href="mailto:sergii.romantsov@gmail.com" target="_blank">sergii.romantsov@gmail.com</a>> wrote:<br>
> ><br>
> > While building of 32bit Mesa gcc doesn't specifies __SSE__ by default.<br>
> > So it has to be done manually by flag '-msee'.<br>
> > Added support of such specification to build-system.<br>
> ><br>
> > CC: Dylan Baker <<a href="mailto:dylan@pnwbakers.com" target="_blank">dylan@pnwbakers.com</a>><br>
> > Bugzilla: <a href="https://bugs.freedesktop.org/show_bug.cgi?id=108560" rel="noreferrer" target="_blank">https://bugs.freedesktop.org/show_bug.cgi?id=108560</a><br>
> > Signed-off-by: Sergii Romantsov <<a href="mailto:sergii.romantsov@globallogic.com" target="_blank">sergii.romantsov@globallogic.com</a>><br>
> <br>
> We can't do that.<br>
> <br>
> Maybe I'm missing some context. What problem are we solving?<br>
<br>
That we don't get fast paths in rounding.h by default on i686, I think?<br>
_______________________________________________<br>
mesa-dev mailing list<br>
<a href="mailto:mesa-dev@lists.freedesktop.org" target="_blank">mesa-dev@lists.freedesktop.org</a><br>
<a href="https://lists.freedesktop.org/mailman/listinfo/mesa-dev" rel="noreferrer" target="_blank">https://lists.freedesktop.org/mailman/listinfo/mesa-dev</a><br>
</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div dir="ltr">Sergii Romantsov<br></div><div dir="ltr"><div style="margin:0px;padding:0px;color:rgb(23,43,77);font-family:-apple-system,BlinkMacSystemFont,"Segoe UI",Roboto,Oxygen,Ubuntu,"Fira Sans","Droid Sans","Helvetica Neue",sans-serif;font-size:14px">GlobalLogic Inc.<br><a href="http://www.globallogic.com/" rel="nofollow" style="color:rgb(0,82,204)" target="_blank">www.globallogic.com</a><br></div></div></div></div></div></div>