Re: [Mesa-dev] [PATCH 2/3] glsl: clone builtin function constants

2017-08-09 Thread Nicolai Hähnle
On 04.08.2017 09:25, Timothy Arceri wrote: f81ede469910d fixed a problem with shaders including IR that was owned by builtins. However the approach of cloning the whole function each time we referenced it lead to a significant reduction in the GLSL IR compiler performance. Everything was

[Mesa-dev] [PATCH 2/3] glsl: clone builtin function constants

2017-08-04 Thread Timothy Arceri
f81ede469910d fixed a problem with shaders including IR that was owned by builtins. However the approach of cloning the whole function each time we referenced it lead to a significant reduction in the GLSL IR compiler performance. Everything was already cloned when inlining the function, as far