Re: [Mesa-dev] [PATCH] anv/pipeline: Move multiple shaders/module finishme

2016-11-04 Thread Jason Ekstrand
I'm pretty sure we can actually just delete the finishme. There are CTS tests that do multiple shaders per module and we pass them. --Jason On Fri, Nov 4, 2016 at 8:34 AM, Robert Bragg wrote: > The heuristic expecting the entrypoint to be named 'main' was causing > false warnings for modules wi

[Mesa-dev] [PATCH] anv/pipeline: Move multiple shaders/module finishme

2016-11-04 Thread Robert Bragg
The heuristic expecting the entrypoint to be named 'main' was causing false warnings for modules with only a single shader that happen to use another name. We now count entrypoints before triggering this warning. Signed-off-by: Robert Bragg --- src/compiler/spirv/spirv_to_nir.c | 6 ++ src/c