On Fri, May 13, 2016 at 12:07 AM, Pohjolainen, Topi < topi.pohjolai...@intel.com> wrote:
> On Wed, May 11, 2016 at 11:37:10AM -0700, Jason Ekstrand wrote: > > The helper was initially created to allow us to set reasonable defaults > as > > we mutated the brw_blorp_prog_data structure in preparation for NIR. Now > > that everything is going through brw_blorp_compile_nir_shader() which > fully > > fills out the brw_blorp_prog_data structure, we don't need the helper. > > --- > > src/mesa/drivers/dri/i965/brw_blorp.c | 16 ---------------- > > src/mesa/drivers/dri/i965/brw_blorp.h | 2 -- > > src/mesa/drivers/dri/i965/brw_blorp_blit.cpp | 8 ++------ > > src/mesa/drivers/dri/i965/brw_blorp_clear.cpp | 2 -- > > 4 files changed, 2 insertions(+), 26 deletions(-) > > Could this be squashed into patch 17? > No. It existed precicely to keep the old blorp code working while we added fields needed for NIR shaders.
_______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev
