On 08/08/17 02:52, Thomas Helland wrote:
Patch 7 has my RB. I'll look closer at the rest this evening.
There's also the stringbuffer work that I have lying that should help.
I just have to find out how to run the gtest tests to verify their working.

oh right. Sorry for the overlap :(

We could commit this first just in case the stringbuffer needs to be rolled back for some reason, as it is a more invasive change.


On Aug 7, 2017 04:19, "Timothy Arceri" <tarc...@itsqueeze.com <mailto:tarc...@itsqueeze.com>> wrote:

    I've sent all but the last 2 already but they haven't received much
    feedback yet.

    This series reduces compile times of the Deus Ex shaders on my
    Ryzen 1800X from 2m27s -> 2m8s with a cold cache on radeonsi.

    Note: The above times are from compiling the shaders with shader-db
    on a single thread, starting the actual game results in around
    double the time/improvement.

    The aim of the series was to target overhead in the parser/AST
    sections of the compiler. I believe there is probably still more
    low hanging fruit in the AST code. I've been profiling this by
    compiling the shaders once and then deleting the index file from
    the cache.

    Please review.

    _______________________________________________
    mesa-dev mailing list
    mesa-dev@lists.freedesktop.org <mailto:mesa-dev@lists.freedesktop.org>
    https://lists.freedesktop.org/mailman/listinfo/mesa-dev
    <https://lists.freedesktop.org/mailman/listinfo/mesa-dev>


_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to