diff options
| author | Timothy Arceri <[email protected]> | 2017-08-09 13:34:07 +1000 |
|---|---|---|
| committer | Timothy Arceri <[email protected]> | 2017-08-11 15:44:15 +1000 |
| commit | 9d41ec21826b1c19da69a183b3c0ce535bfd7b10 (patch) | |
| tree | d4033f866db74362976588119aca1ba18a3ea0d0 /src/intel/compiler | |
| parent | 77f5221233ea427b622af46831feed438e0dd59e (diff) | |
glsl: stop cloning builtin fuctions _mesa_glsl_find_builtin_function()
The cloning was introduced in f81ede469910d to fix 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 Ryzen 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%.
Tested-by: Dieter Nützel <[email protected]>
Reviewed-by: Kenneth Graunke <[email protected]>
Reviewed-by: Nicolai Hähnle <[email protected]>
Diffstat (limited to 'src/intel/compiler')
0 files changed, 0 insertions, 0 deletions