[FFmpeg-devel] [PATCH] vvcdec: add TODO for combining transform, lmcs_scale_chroma, and add_residual

[FFmpeg-devel] [PATCH] vvcdec: add TODO for combining transform, lmcs_scale_chroma, and add_residual

[Nuo Mi]

Thanks for the suggestion from Lynne.
---
 libavcodec/vvc/vvc_intra.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/libavcodec/vvc/vvc_intra.c b/libavcodec/vvc/vvc_intra.c
index 43de312a71..214ad38c8c 100644
--- a/libavcodec/vvc/vvc_intra.c
+++ b/libavcodec/vvc/vvc_intra.c
@@ -552,6 +552,8 @@ static void itransform(VVCLocalContext *lc, TransformUnit *tu, const int tu_idx,
 
             if (chroma_scale)
                 fc->vvcdsp.intra.lmcs_scale_chroma(lc, temp, tb->coeffs, w, h, cu->x0, cu->y0);
+            // TODO: Address performance issue here by combining transform, lmcs_scale_chroma, and add_residual into one function.
+            // Complete this task before implementing ASM code.
             fc->vvcdsp.itx.add_residual(dst, chroma_scale ? temp : tb->coeffs, w, h, stride);
 
             if (tu->joint_cbcr_residual_flag && tb->c_idx)
-- 
2.25.1

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-devel] [PATCH] vvcdec: add TODO for combining transform, lmcs_scale_chroma, and add_residual

[Lynne]

Jan 3, 2024, 16:53 by nuomi2021@gmail.com:

> Thanks for the suggestion from Lynne.
> ---
>  libavcodec/vvc/vvc_intra.c | 2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/libavcodec/vvc/vvc_intra.c b/libavcodec/vvc/vvc_intra.c
> index 43de312a71..214ad38c8c 100644
> --- a/libavcodec/vvc/vvc_intra.c
> +++ b/libavcodec/vvc/vvc_intra.c
> @@ -552,6 +552,8 @@ static void itransform(VVCLocalContext *lc, TransformUnit *tu, const int tu_idx,
>  
>  if (chroma_scale)
>  fc->vvcdsp.intra.lmcs_scale_chroma(lc, temp, tb->coeffs, w, h, cu->x0, cu->y0);
> +            // TODO: Address performance issue here by combining transform, lmcs_scale_chroma, and add_residual into one function.
> +            // Complete this task before implementing ASM code.
>  fc->vvcdsp.itx.add_residual(dst, chroma_scale ? temp : tb->coeffs, w, h, stride);
>  
>  if (tu->joint_cbcr_residual_flag && tb->c_idx)
>

LGTM
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-devel] [PATCH] vvcdec: add TODO for combining transform, lmcs_scale_chroma, and add_residual

[Nuo Mi]

On Sun, Jan 7, 2024 at 4:03 AM Lynne <dev@lynne.ee> wrote:

> Jan 3, 2024, 16:53 by nuomi2021@gmail.com:
>
> > Thanks for the suggestion from Lynne.
> > ---
> >  libavcodec/vvc/vvc_intra.c | 2 ++
> >  1 file changed, 2 insertions(+)
> >
> > diff --git a/libavcodec/vvc/vvc_intra.c b/libavcodec/vvc/vvc_intra.c
> > index 43de312a71..214ad38c8c 100644
> > --- a/libavcodec/vvc/vvc_intra.c
> > +++ b/libavcodec/vvc/vvc_intra.c
> > @@ -552,6 +552,8 @@ static void itransform(VVCLocalContext *lc,
> TransformUnit *tu, const int tu_idx,
> >
> >  if (chroma_scale)
> >  fc->vvcdsp.intra.lmcs_scale_chroma(lc, temp, tb->coeffs, w, h, cu->x0,
> cu->y0);
> > +            // TODO: Address performance issue here by combining
> transform, lmcs_scale_chroma, and add_residual into one function.
> > +            // Complete this task before implementing ASM code.
> >  fc->vvcdsp.itx.add_residual(dst, chroma_scale ? temp : tb->coeffs, w,
> h, stride);
> >
> >  if (tu->joint_cbcr_residual_flag && tb->c_idx)
> >
>
> LGTM
>
Thank you, Lynne.
Pushed

> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel@ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".
>
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".