[Mesa-dev] [PATCH v2 1/2] gm107/ir: AL2P writes to a predicate register

Samuel Pitoiset samuel.pitoiset at gmail.com
Sat Sep 10 17:31:09 UTC 2016


This series is:

Reviewed-by: Samuel Pitoiset <samuel.pitoiset at gmail.com>

On 09/10/2016 06:58 PM, Ilia Mirkin wrote:
> We have to force it to write to predicate 7 (aka PT) in order for it not
> to mess up another predicate. Unclear what would be returned in the
> predicate, perhaps an error code for out-of-bounds requests. Blob
> doesn't seem to check it.
>
> Signed-off-by: Ilia Mirkin <imirkin at alum.mit.edu>
> Cc: mesa-stable at lists.freedesktop.org
> ---
>  src/gallium/drivers/nouveau/codegen/nv50_ir_emit_gm107.cpp | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/src/gallium/drivers/nouveau/codegen/nv50_ir_emit_gm107.cpp b/src/gallium/drivers/nouveau/codegen/nv50_ir_emit_gm107.cpp
> index 791273f..cfde66c 100644
> --- a/src/gallium/drivers/nouveau/codegen/nv50_ir_emit_gm107.cpp
> +++ b/src/gallium/drivers/nouveau/codegen/nv50_ir_emit_gm107.cpp
> @@ -2307,6 +2307,7 @@ CodeEmitterGM107::emitAL2P()
>  {
>     emitInsn (0xefa00000);
>     emitField(0x2f, 2, (insn->getDef(0)->reg.size / 4) - 1);
> +   emitPRED (0x2c);
>     emitO    (0x20);
>     emitField(0x14, 11, insn->src(0).get()->reg.data.offset);
>     emitGPR  (0x08, insn->src(0).getIndirect(0));
>


More information about the mesa-dev mailing list