On Fri, Mar 17, 2017 at 10:10 AM, Emil Velikov <[email protected]> wrote:
> 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 :-\ > Put it on the shelf rather than the bin please. :-) I think we're probably going to be deleting the AUB annotation stuff from i965 before too much longer.
_______________________________________________ mesa-dev mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/mesa-dev
