On Mon, Aug 10, 2015 at 09:09:41AM -0700, Jason Ekstrand wrote: > On Aug 10, 2015 4:14 AM, "Neil Roberts" <n...@linux.intel.com> wrote: > > > > If we go with this patch perhaps it would be good to remove > > supports_simd16_3src entirely from brw_device_info and any code that is > > referring to it in order to avoid carrying around useless code. > > Currently it seems like it would be quite easy to add a new > > brw_device_info and forget to add supports_simd16_3src and never notice > > that it is redundantly using the fallback. > > I would second that. >
Either of you opposed to doing it as a follow-on patch? (This failure you speak of was the primary goal for removing it). Initially, I tried to remove it, but we do still seem to use this logic pre-HSW, and so things got messier than I initially anticipated. Neil, you said "if we go with this patch" - I haven't heard any opposition, do you have any? [snip] _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev