[Mesa-dev] [Mesa-stable] [PATCH] st/mesa: set result writemask based on ir type

Ilia Mirkin imirkin at alum.mit.edu
Tue Mar 14 13:16:37 UTC 2017


I am waiting to have time to do proper testing on the patch like I
promised. Thus far, the time to do so has failed to materialize.

On Mar 14, 2017 8:27 AM, "Emil Velikov" <emil.l.velikov at gmail.com> wrote:

> Ilia, are you waiting for another person to review the patch or it's
> no longer applicable ?
>
> Thanks
> Emil
>
> On 10 March 2017 at 11:07, Marek Olšák <maraeo at gmail.com> wrote:
> > Reviewed-by: Marek Olšák <marek.olsak at amd.com>
> >
> > Marek
> >
> > On Sat, Mar 4, 2017 at 7:52 PM, Ilia Mirkin <imirkin at alum.mit.edu>
> wrote:
> >> This prevents textureQueryLevels, which maps as LODQ, from ending up
> >> with a xyzw writemask, which is illegal.
> >>
> >> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=100061
> >> Signed-off-by: Ilia Mirkin <imirkin at alum.mit.edu>
> >> Cc: mesa-stable at lists.freedesktop.org
> >> ---
> >>
> >> Untested beyond making sure that the shader in the above bug ends up
> with
> >> a .xy writemask. I don't have access to a reasonable testing rig right
> now,
> >> would be ideal if someone could give it a run through theirs...
> >>
> >>  src/mesa/state_tracker/st_glsl_to_tgsi.cpp | 1 +
> >>  1 file changed, 1 insertion(+)
> >>
> >> diff --git a/src/mesa/state_tracker/st_glsl_to_tgsi.cpp
> b/src/mesa/state_tracker/st_glsl_to_tgsi.cpp
> >> index af41bdb..63b681f 100644
> >> --- a/src/mesa/state_tracker/st_glsl_to_tgsi.cpp
> >> +++ b/src/mesa/state_tracker/st_glsl_to_tgsi.cpp
> >> @@ -4165,6 +4165,7 @@ glsl_to_tgsi_visitor::visit(ir_texture *ir)
> >>      */
> >>     result_src = get_temp(ir->type);
> >>     result_dst = st_dst_reg(result_src);
> >> +   result_dst.writemask = (1 << ir->type->vector_elements) - 1;
> >>
> >>     switch (ir->op) {
> >>     case ir_tex:
> >> --
> >> 2.10.2
> >>
> >> _______________________________________________
> >> mesa-stable mailing list
> >> mesa-stable at lists.freedesktop.org
> >> https://lists.freedesktop.org/mailman/listinfo/mesa-stable
> > _______________________________________________
> > mesa-stable mailing list
> > mesa-stable at lists.freedesktop.org
> > https://lists.freedesktop.org/mailman/listinfo/mesa-stable
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20170314/b415cb46/attachment-0001.html>


More information about the mesa-dev mailing list