Re: [Mesa-dev] [PATCH 07/12] glcpp: Skip unnecessary line continuations removal

2017-01-18 Thread Ian Romanick
This patch is Reviewed-by: Ian Romanick On 01/07/2017 11:02 AM, Vladislav Egorov wrote: > Overwhelming majority of shaders don't use line continuations. In my > shader-db only shaders from the Talos Principle and Serious Sam used > them, less than 1% out of all shaders. Optimize for this case, d

[Mesa-dev] [PATCH 07/12] glcpp: Skip unnecessary line continuations removal

2017-01-07 Thread Vladislav Egorov
Overwhelming majority of shaders don't use line continuations. In my shader-db only shaders from the Talos Principle and Serious Sam used them, less than 1% out of all shaders. Optimize for this case, don't do any copying if no line continuation was found. --- src/compiler/glsl/glcpp/pp.c | 10 +++