[Nouveau] [PATCH v2 1/4] nvc0/ir: clear subop when folding constant expressions

Ilia Mirkin imirkin at alum.mit.edu
Tue Jun 3 14:06:08 PDT 2014


On Tue, Jun 3, 2014 at 4:58 PM, Tobias Klausmann
<tobias.johannes.klausmann at mni.thm.de> wrote:
> Some operations (e.g. OP_MUL/OP_MAD/OP_EXTBF might have a subop set.
> After folding, make sure that it is cleared
>
> Signed-off-by: Tobias Klausmann <tobias.johannes.klausmann at mni.thm.de>

Reviewed-by: Ilia Mirkin <imirkin at alum.mit.edu>

> ---
>  src/gallium/drivers/nouveau/codegen/nv50_ir_peephole.cpp | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/src/gallium/drivers/nouveau/codegen/nv50_ir_peephole.cpp b/src/gallium/drivers/nouveau/codegen/nv50_ir_peephole.cpp
> index 1a2c2e6..58092f4 100644
> --- a/src/gallium/drivers/nouveau/codegen/nv50_ir_peephole.cpp
> +++ b/src/gallium/drivers/nouveau/codegen/nv50_ir_peephole.cpp
> @@ -563,6 +563,7 @@ ConstantFolding::expr(Instruction *i,
>     } else {
>        i->op = i->saturate ? OP_SAT : OP_MOV; /* SAT handled by unary() */
>     }
> +   i->subOp = 0;
>  }
>
>  void
> --
> 1.8.4.5
>


More information about the Nouveau mailing list