On Wed, Oct 07, 2015 at 07:11:40AM -0700, Kristian H?gsberg Kristensen wrote: > This series introduces a new libtool convenience library, > libi965_compiler.la. Most of the series is concerned with moving bits > of code around to allow for splitting the compiler files in the > second-to-last patch. > > The end result is still not completely ideal - we still have a bunch > of brw_context references in there, INTEL_DEUBG is defined in the > compiler library and other oddities. However, the split lets us link > the compiler unit tests to just libi965_compiler.la and drop a few > dependencies there and of course, lets us use the compiler in other > projects more easily. > > Kristian Høgsberg Kristensen (12): > i965: Move brw_dump_ir() out of brw_*_emit() functions > util: Move DRI parse_debug_string() to util > i965: Configure bufmgr debug options from intel_screen.c > i965: Move brw_link_shader() and friends to new file brw_link.cpp > i965/cs: Split out helper for building local id payload > i965: Move brw_mark_surface_used() to brw_shader.cpp > i965: Use util_next_power_of_two() for brw_get_scratch_size() > i965: Move brw_select_clip_planes() to brw_shader.cpp
A few minor comments that you can take into account if you like, but patches 1-8 are: Reviewed-by: Topi Pohjolainen <topi.pohjolai...@intel.com> _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev
