[Mesa-dev] [PATCH v3 00/13] TGSI: improved live range tracking, also including arrays

Gert Wollny gw.fossdev at gmail.com
Sun Apr 29 07:55:00 UTC 2018


Hello Benedikt, 

Am Sonntag, den 29.04.2018, 00:06 +0200 schrieb Benedikt Schemmer:
> Hi Gert
> 
> Am 28.04.2018 um 23:51 schrieb Gert Wollny:
> > 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.
> > 
> 
> Ah, ok. Was wondering why your debug code doesn't output anything.
> Makes sense now ;)
Not exactly, the reason there is no output is because -DNDEBUG is set.
Without it the statistics should also be printed out on radeonsi, but
thinking of it I should probably disable it when register_merge is not
accessed, because without this the numbers will be inflated and don't
have much meaning.

> So is this useless on radeonsi?
Indeed. 

> Seemed interesting to me.
:) it certainly helps on r600 


> > > compile times went up though:
> > 
> > This is strange, because "see above". Did you compile with debug
> > information and c++11 or higher enables?
...
> >   
> > 
> not intentionally:

Then you should actually not run any code that this series adds to
mesa. I checked again, apart from the debugging output nothing will
ever be run if a drivers that report
PIPE_SHADER_CAP_TGSI_SKIP_MERGE_REGISTERS != 0 (as does radeonsi). 

Best, 
Gert 


More information about the mesa-dev mailing list