[Mesa-dev] [PATCH v3 00/13] TGSI: improved live range tracking, also including arrays
Gert Wollny
gw.fossdev at gmail.com
Sat Apr 28 21:51:21 UTC 2018
Am Samstag, den 28.04.2018, 22:43 +0200 schrieb Benedikt Schemmer:
> The patches apply cleanly, however I just did a shader-db test run
> and can't find a difference with your patch
> applied, am I doing something wrong?
AFAIK radeonsi doesn't use the register-merge optimizer in TGSI.
>
> compile times went up though:
This is strange, because "see above". Did you compile with debug
information and c++11 or higher enables? In this case there is one
access to a static variable (first patch) that per c++11 standard
should be thread save, which means that there might be a mutex
protecting access to that variable, and this would explain the longer
runtime in a multi-threaded environment.
Best,
Gert
>
> before:
> Thread 3 took 113.72 seconds and compiled 17899 shaders (not
> including SIMD16) with 2232 GL context switches
> Thread 5 took 113.23 seconds and compiled 17767 shaders (not
> including SIMD16) with 2150 GL context switches
> Thread 7 took 116.63 seconds and compiled 18030 shaders (not
> including SIMD16) with 2219 GL context switches
> Thread 1 took 117.10 seconds and compiled 17966 shaders (not
> including SIMD16) with 2154 GL context switches
> Thread 4 took 113.76 seconds and compiled 18097 shaders (not
> including SIMD16) with 2285 GL context switches
> Thread 2 took 113.61 seconds and compiled 17111 shaders (not
> including SIMD16) with 1934 GL context switches
> Thread 6 took 118.93 seconds and compiled 17887 shaders (not
> including SIMD16) with 2205 GL context switches
> Thread 0 took 112.91 seconds and compiled 18232 shaders (not
> including SIMD16) with 2321 GL context switches
>
> with your patch:
> Thread 1 took 119.41 seconds and compiled 18495 shaders (not
> including SIMD16) with 2237 GL context switches
> Thread 7 took 122.11 seconds and compiled 17228 shaders (not
> including SIMD16) with 2105 GL context switches
> Thread 4 took 120.57 seconds and compiled 17989 shaders (not
> including SIMD16) with 2165 GL context switches
> Thread 5 took 119.79 seconds and compiled 17709 shaders (not
> including SIMD16) with 2190 GL context switches
> Thread 6 took 121.95 seconds and compiled 17804 shaders (not
> including SIMD16) with 2209 GL context switches
> Thread 2 took 121.43 seconds and compiled 17819 shaders (not
> including SIMD16) with 2167 GL context switches
> Thread 0 took 117.16 seconds and compiled 18180 shaders (not
> including SIMD16) with 2151 GL context switches
> Thread 3 took 124.79 seconds and compiled 17765 shaders (not
> including SIMD16) with 2176 GL context switches
>
>
>
> Radeon RX 560 Series (POLARIS11, DRM 3.26.0, 4.17.0-rc2+, LLVM 6.0.0)
>
> Mesa git of today
>
> PERCENTAGE DELTAS Shaders SGPRs VGPRs SpillSGPR
> SpillVGPR PrivVGPR Scratch CodeSize MaxWaves Waits
> 0ad 6 . . . .
> . . . . .
> aer 590 . . . .
> . . . . .
> alien_isolation 1414 . . . .
> . . . . .
> anholt 10 . . . .
> . . . . .
> bioshock_infinite 2581 . . . .
> . . . . .
> blackmesa 584 . . . .
> . . . . .
> cat 573 . . . .
> . . . . .
> csgo 1392 . . . .
> . . . . .
> deadisland_definitive 1776 . . . .
> . . . . .
> deadisland_original 11602 . . . .
> . . . . .
> deadisland_riptide_.. 293 . . . .
> . . . . .
> deusex_mankind 5051 . . . .
> . . . . .
> dirtrally 787 . . . .
> . . . . .
> dolphin 22 . . . .
> . . . . .
> dyinglight 4012 . . . .
> . . . . .
> eurotruck2 216 . . . .
> . . . . .
> f1_2015 746 . . . .
> . . . . .
> glamor 16 . . . .
> . . . . .
> hl2ep1 294 . . . .
> . . . . .
> hl2ep2 154 . . . .
> . . . . .
> hl2lostcoast 66 . . . .
> . . . . .
> hlsl3 582 . . . .
> . . . . .
> humus-
> celshading 4 . . . . .
> . . . .
> humus-
> domino 6 . . . . .
> . . . .
> humus-
> dynamicbranching 24 . . . . .
> . . . .
> humus-
> hdr 10 . . . . .
> . . . .
> humus-
> portals 2 . . . . .
> . . . .
> humus-
> volumetricfog.. 6 . . . . .
> . . . .
> kerbal 1016 . . . .
> . . . . .
> larago 664 . . . .
> . . . . .
> madmax 354 . . . .
> . . . . .
> metro2033redux 4410 . . . .
> . . . . .
> nexuiz 80 . . . .
> . . . . .
> piglit 80734 . . . .
> . . . . .
> ruiner 685 . . . .
> . . . . .
> sauerbraten 7 . . . .
> . . . . .
> serioussam2017 736 . . . .
> . . . . .
> soma 436 . . . .
> . . . . .
> specops 1814 . . . .
> . . . . .
> stellaris 434 . . . .
> . . . . .
> supertuxkart 4 . . . .
> . . . . .
> talos 762 . . . .
> . . . . .
> tesseract 430 . . . .
> . . . . .
> tombraider 1012 . . . .
> . . . . .
> total_war_shogun_2 176 . . . .
> . . . . .
> total_war_warhammer 218 . . . .
> . . . . .
> ubershaders 54 . . . .
> . . . . .
> ug_gettysburg 149 . . . .
> . . . . .
> unigine_heaven 226 . . . .
> . . . . .
> unigine_superposition 733 . . . .
> . . . . .
> unigine_valley 288 . . . .
> . . . . .
> unity 72 . . . .
> . . . . .
> w40kdawn2 421 . . . .
> . . . . .
> w40kdawn3 164 . . . .
> . . . . .
> warsow 176 . . . .
> . . . . .
> warzone2100 4 . . . .
> . . . . .
> witcher2 928 . . . .
> . . . . .
> x3_albion 641 . . . .
> . . . . .
> xblades 208 . . . .
> . . . . .
> xcom 1020 . . . .
> . . . . .
> xcom2 1439 . . . .
> . . . . .
> yofrankie 82 . . . .
> . . . . .
> ------------------------------------------------------------------
> ----------------------------------------------------
> All
> affected 0 . . . .
> . . . . .
> ------------------------------------------------------------------
> ----------------------------------------------------
> Total 133396 . . . .
> . . . . .
More information about the mesa-dev
mailing list