On 17 March 2017 at 16:55, Emil Velikov <[email protected]> wrote: > From: Emil Velikov <[email protected]> > > The enum aub_state_struct_type type is a seeming left over from the > i965 days. Since we don't use we can safely drop it. > > As a nice side-effect the only libdrm_intel requirement in src/intel > [outside of tools/aubinator] - include intel_aub.h can now be removed. > And with next commit we'll remove the dependency in the build system(s). > > Signed-off-by: Emil Velikov <[email protected]> > --- > src/intel/blorp/blorp_genX_exec.h | 12 +++++------- > src/intel/vulkan/genX_blorp_exec.c | 1 - > 2 files changed, 5 insertions(+), 8 deletions(-) > Hmm actually this won't fly since we have the declaration in blorp, and two distinct definitions - anv and i965. With the latter actually using the type to annotate appropriately.
We could drop all that, but I'm not sure how many/often people use DEBUG_BATCH/do_batch_dump. So unless someone voices for the removal of do_batch_dump (and friends) this series goes in the bin :-\ Emil _______________________________________________ mesa-dev mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/mesa-dev
