Patch looks good to me, but I think the signal changes should be in their own commit. On 7 February 2017 at 16:12, Maarten Lankhorst <maarten.lankhorst@xxxxxxxxxxxxxxx> wrote: ... > +#define VG(x) x > +#else > +#define VG(x) > +#endif Wouldn't be better to stub VALGRIND_MAKE_MEM_DEFINED instead? Regards, Tomeu _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx