On Tuesday, August 8, 2017 8:34:07 PM PDT Timothy Arceri wrote: > The cloning was introduced in f81ede469910d to fixed a problem with > shaders including IR that was owned by builtins. > > However the approach of cloning the whole function each time we > reference a builtin lead to a significant reduction in the GLSL > IR compilers performance. > > The previous patch fixes the ownership problem in a more precise > way. So we can now remove this cloning. > > Testing on a Ryzan 7 1800X shows a ~15% decreases in compiling the > Deus Ex: Mankind Divided shaders on radeonsi (which take 5min+ on > some machines). Looking just at the GLSL IR compiler the speed up > is ~40%. > > Cc: Kenneth Graunke <kenn...@whitecape.org> > Cc: Lionel Landwerlin <lionel.g.landwer...@intel.com> > > Tested-by: Dieter Nützel <die...@nuetzel-hh.de>
With 5v2, this is: Reviewed-by: Kenneth Graunke <kenn...@whitecape.org> Thanks!
signature.asc
Description: This is a digitally signed message part.
_______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev
