[Mesa-dev] [Mesa-stable] [PATCH 3/3] i965/vec4: don't modify regioning parameters to the sources of DF align1 instructions

Francisco Jerez currojerez at riseup.net
Fri Apr 28 23:33:51 UTC 2017


Samuel Iglesias Gonsálvez <siglesias at igalia.com> writes:

> The regioning parameters are now properly set by convert_to_hw_regs()
> and we don't need to fix them in the generator.
>

It would be worth stressing here that the "fix" previously done in the
generator was strictly speaking wrong for any non-identity regions.
With that clarified patch is:

Reviewed-by: Francisco Jerez <currojerez at riseup.net>

> Signed-off-by: Samuel Iglesias Gonsálvez <siglesias at igalia.com>
> Cc: "17.1" <mesa-stable at lists.freedesktop.org>
> ---
>  src/intel/compiler/brw_vec4_generator.cpp | 9 +--------
>  1 file changed, 1 insertion(+), 8 deletions(-)
>
> diff --git a/src/intel/compiler/brw_vec4_generator.cpp b/src/intel/compiler/brw_vec4_generator.cpp
> index e786ac6a0ca..753b00c4ed1 100644
> --- a/src/intel/compiler/brw_vec4_generator.cpp
> +++ b/src/intel/compiler/brw_vec4_generator.cpp
> @@ -1980,8 +1980,6 @@ generate_code(struct brw_codegen *p,
>           else
>              spread_dst = stride(dst, 8, 4, 2);
>  
> -         src[0].vstride = BRW_VERTICAL_STRIDE_4;
> -         src[0].width = BRW_WIDTH_4;
>           brw_MOV(p, spread_dst, src[0]);
>  
>           brw_set_default_access_mode(p, BRW_ALIGN_16);
> @@ -2016,9 +2014,7 @@ generate_code(struct brw_codegen *p,
>           src[0] = retype(src[0], BRW_REGISTER_TYPE_UD);
>           if (inst->opcode == VEC4_OPCODE_PICK_HIGH_32BIT)
>              src[0] = suboffset(src[0], 1);
> -         src[0].vstride = BRW_VERTICAL_STRIDE_8;
> -         src[0].width = BRW_WIDTH_4;
> -         src[0].hstride = BRW_HORIZONTAL_STRIDE_2;
> +         src[0] = spread(src[0], 2);
>           brw_MOV(p, dst, src[0]);
>  
>           brw_set_default_access_mode(p, BRW_ALIGN_16);
> @@ -2041,9 +2037,6 @@ generate_code(struct brw_codegen *p,
>           dst.hstride = BRW_HORIZONTAL_STRIDE_2;
>  
>           src[0] = retype(src[0], BRW_REGISTER_TYPE_UD);
> -         src[0].vstride = BRW_VERTICAL_STRIDE_4;
> -         src[0].width = BRW_WIDTH_4;
> -         src[0].hstride = BRW_HORIZONTAL_STRIDE_1;
>           brw_MOV(p, dst, src[0]);
>  
>           brw_set_default_access_mode(p, BRW_ALIGN_16);
> -- 
> 2.11.0
>
> _______________________________________________
> mesa-stable mailing list
> mesa-stable at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-stable
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 212 bytes
Desc: not available
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20170428/d24d1b6b/attachment.sig>


More information about the mesa-dev mailing list