On Tuesday, February 28, 2017 9:03:12 PM PST Jason Ekstrand wrote: > This little series moves the i965 back-end compiler to src/intel/compiler. > It's not incredibly pretty; I didn't do as much header clean-up as perhaps > could have been done. Anything that I didn't know what to do with I moved > to brw_shader or brw_program. The second-to-last patch which shuffles the > header includes around is also one big patch. It could be split up but > there didn't seem to be much of a point. > > Jason Ekstrand (24): > i965: Move a couple of #defines from brw_context to brw_compiler > i965: Move some helpers from brw_context.h to brw_shader.h > i965: Move brw_disassemble_inst to brw_eu.h > i965: Move some gen4 WM defines to brw_compiler.h > i965: Move assign_common_binding_table_offsets to brw_program > i965/vue_map: Stop using GLbitfield types > i965: Get rid of BRW_PRIM_OFFSET > i965: Don't use MAX_SURFACES in mark_surface_used > i965/gs: Add the gl_prim_to_hw_prim table to vec4_gs_visitor.cpp > i964/gs: Move MAX_GS_INPUT_VERTICES to brw_vec4_gs_visitor.h > i965: Move SOL binding #defines to brw_compiler.h > i965: Move SHADER_TIME_STRIDE to brw_compiler.h > i965: Move brw_register_blocks to brw_fs.cpp > i965/inst: Stop using fi_type > i965: Move BRW_MAX_DRAW_BUFFERS to brw_compiler.h > i965: Move BRW_ATTRIB_WA_* defines to brw_compiler.h > i965: Make mark_surface_used a static inline in brw_compiler.h > i965: Move channel_expressions and vector_splitting to brw_program.h > i965: Move image uniform setup to brw_nir_uniforms.cpp > i965: Move a bunch of pre-compile and link stuff to brw_program.h > i965: Add a header for brw_vec4_vs_visitor > i965: Delete brw_do_cubemap_normalize > i965: Reduce cross-pollination between the DRI driver and compiler > i965: Move the back-end compiler to src/intel/compiler
Assuming you take my suggestion on patch 21...patches 1-22 are: Reviewed-by: Kenneth Graunke <kenn...@whitecape.org> and patch 23 is: Acked-by: Kenneth Graunke <kenn...@whitecape.org> I believe Matt asked you to hold off on moving the compiler earlier today, so I'm a bit surprised to see this. It probably makes sense to land the precursor patches - I doubt they'll conflict too badly with any of the outstanding work. But please get an ack from him before pushing anything.
signature.asc
Description: This is a digitally signed message part.
_______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev
