<html> <head> <base href="https://bugs.freedesktop.org/"> </head> <body> <div> <a class="bz_bug_link bz_status_NEW " title="NEW - 4.5% perf drop in CSDof with "nir: Optimize integer division and modulus with 1"" href="https://bugs.freedesktop.org/show_bug.cgi?id=98409#c5">Comment # 5</a> on <a class="bz_bug_link bz_status_NEW " title="NEW - 4.5% perf drop in CSDof with "nir: Optimize integer division and modulus with 1"" href="https://bugs.freedesktop.org/show_bug.cgi?id=98409">bug 98409</a> from <a class="email" href="mailto:jljusten@gmail.com" title="Jordan Justen <jljusten@gmail.com>"> Jordan Justen</a> <pre>(In reply to Eero Tamminen from <a href="show_bug.cgi?id=98409#c4">comment #4</a>) > Confirming Matt's observation. > > Reverting the indicated change from Mesa HEAD, doesn't affect the > performance, although going back to version before this commit (or commits > before it), shows 4-5% better perf than the version built from the commit. > I.e. additional changes after the indicated commit also have effect on CSDof > test. When reverting: 4d35683 nir: Optimize integer division and modulus with 1 Did you also revert these? 64c3d73 i965/cs: Don't use a thread channel ID for small local sizes 1fa000a i965/cs: Use udiv/umod for local IDs It would be good to know if taking the current master, and reverting these 3 patches regains the 4-5%.</pre> </div> <hr> You are receiving this mail because: <ul> <li>You are the QA Contact for the bug.</li> </ul> </body> </html>