On Thursday, June 26, 2014 12:53:11 PM Ilia Mirkin wrote: > On Sun, Jun 22, 2014 at 1:49 PM, Ilia Mirkin <imir...@alum.mit.edu> wrote: > > All of the bits appear to already be in place to support this in the > > sampler (which the original AMD version didn't allow). > > > > Signed-off-by: Ilia Mirkin <imir...@alum.mit.edu> > > --- > > > > I'm probably missing some reason why this wasn't already enabled, but sending > > this out anyways. Feel free to point out the error in my reasoning :) > > Actualy I noticed a semi-similar patch was sent out nearly a year ago: > > http://lists.freedesktop.org/archives/mesa-dev/2013-August/043652.html > > To which Ken's response was: > > http://lists.freedesktop.org/archives/mesa-dev/2013-August/043660.html > > after which I guess the ball got dropped... > > Ken, is my patch OK then? Or should I do a s/AMD/ARB/ throughout the > tree as well? Or some third option :) > > -ilia
Yeah, I'm happy to see this land as is. I'd be in favor of s/AMD/ARB/ in a follow-up patch, since the ARB extension is (slight) superset of the AMD one. Reviewed-by: Kenneth Graunke <kenn...@whitecape.org>
signature.asc
Description: This is a digitally signed message part.
_______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev
