[FFmpeg-devel] [PATCH 2/2] lavc/blockdsp: R-V V fill_block

[flow gg]

Since there is no 8x16, I changed m8 to m4, and updated it in the reply



flow gg <hlefthleft at gmail.com> 于2024年4月30日周二 08:26写道：

> Hi, I initially used a loop, but according to libavcodec/blockdsp.h,
>
> the maximum is 8x16 = 128 bytes, so using ff_get_rv_vlenb() >= 16 and m8
> does not require a loop.
>
> ```
> /* add and put pixel (decoding)
>  * Block sizes for op_pixels_func are 8x4,8x8 16x8 16x16.
>  * h for op_pixels_func is limited to { width / 2, width },
>  * but never larger than 16 and never smaller than 4. */
> typedef void (*op_fill_func)(uint8_t *block /* align width (8 or 16) */,
>                              uint8_t value, ptrdiff_t line_size, int h);
> ```
>
> Rémi Denis-Courmont <remi at remlab.net> 于2024年4月30日周二 01:31写道：
>
>> Le maanantaina 29. huhtikuuta 2024, 10.09.41 EEST flow gg a écrit :
>> >
>>
>> Are you sure that this works with all vector lengths?
>> The block8 code looks odd.
>>
>> --
>> レミ・デニ-クールモン
>> http://www.remlab.net/
>> _______________________________________________
>> ffmpeg-devel mailing list
>> ffmpeg-devel at ffmpeg.org
>> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>>
>> To unsubscribe, visit link above, or email
>> ffmpeg-devel-request at ffmpeg.org with subject "unsubscribe".
>>
>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0002-lavc-blockdsp-R-V-V-fill_block.patch
Type: text/x-patch
Size: 2225 bytes
Desc: not available
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20240430/1117aba7/attachment.bin>