From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from ffbox0-bg.ffmpeg.org (ffbox0-bg.ffmpeg.org [79.124.17.100]) by master.gitmailbox.com (Postfix) with ESMTPS id 08CC044B4F for ; Mon, 14 Jul 2025 16:52:51 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.ffmpeg.org (Postfix) with ESMTP id 8BAA868E0FF; Mon, 14 Jul 2025 19:52:47 +0300 (EEST) Received: from mail-yw1-f177.google.com (mail-yw1-f177.google.com [209.85.128.177]) by ffbox0-bg.ffmpeg.org (Postfix) with ESMTPS id 0355B68D820 for ; Mon, 14 Jul 2025 19:52:40 +0300 (EEST) Received: by mail-yw1-f177.google.com with SMTP id 00721157ae682-70e64b430daso46540387b3.3 for ; Mon, 14 Jul 2025 09:52:40 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1752511959; x=1753116759; h=to:subject:message-id:date:from:in-reply-to:references:mime-version :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=i/1ygNQIHMU7cAyH+2VEmav3DjWW/nN3uYvVgEfdLZA=; b=DumWstreFnDTgFA7n4IevgSmo5+xVT/mgiLaWIeNr20eb3eYVgLu0Cx0AZI9uTGpdO ZrzKMoVpeqUrvfgGxQPXMucXI1OYMd53hw/O7dqqJkm8I8rgAmWfNrdde/eXa3TIK4L0 DEyJqNb88H8CBdhMSS9ZzjCJwG4C7xpSxxzx6pzRpWSMfoHSxY2zbHPDy+y+0+PnrdxG F4OXa4UNc0sGqc5uiSCYXqjxqb5/8zkjEsrZn0uHZlvE9RkQFXvQKkDDGlCYFhe7bcov GYqKwajdQjxuf263gSxYhwEmh/pcT3Tr9aWQy7GgaSzot2oT03LX78/lHS3SFEaNvo6M jllQ== X-Gm-Message-State: AOJu0YzQuOHjlKtzoxMF1S/LAoKYYZwNu1BBv3dcwM2jbSh0TYXqSNMD YVn+3wprsnxJHu8e3apoF6owkffZS7+CaBw5nk/rOicoZNwdoF4u6BILRna3CcUqDxht1fot/aB UqwIc9XjYdfd2e3iiLBOxf8u/mu6BzLkyPg== X-Gm-Gg: ASbGncu+v02jDTsvqSm7rsDF/EXTN5A/9dUC5Y3EyZ2Hv7onW6VfY5Kl0RDQNR6zt5m 4WtaQflcPBdTNQJBoWVFM3M5AhXv0OUHmPD0VuVC4x3Uw/XOkLRQfMwuNOzZbm1N5H/0x/IWz4g DkjIXZVpZFHIGtd9SD2GlyUNGP/OpZyZp6KLpJIxTKnCG5dxcQZkskGBb6Ax+aTwoKcvta8wMeD iFNU4k= X-Google-Smtp-Source: AGHT+IFO4fmojuwa9F7RuKbuSwbwlObrjtkkgXrFdMK2vG1cN27r3NlvofZIScfyZW6QqXAjbZTcWJjfDhotLE9PbBY= X-Received: by 2002:a05:690c:a086:10b0:712:cc11:aed with SMTP id 00721157ae682-717d786c579mr133503197b3.2.1752511959360; Mon, 14 Jul 2025 09:52:39 -0700 (PDT) MIME-Version: 1.0 References: <20250713011030.1156550-1-michael@niedermayer.cc> <20250713011030.1156550-3-michael@niedermayer.cc> In-Reply-To: <20250713011030.1156550-3-michael@niedermayer.cc> Date: Mon, 14 Jul 2025 17:49:19 +0100 X-Gm-Features: Ac12FXxKd2chVB9AAWuMgyeVVAhAlsbdvmT4yOpQytazYGgPUhSm5a-YTVPPeMk Message-ID: To: FFmpeg development discussions and patches X-Content-Filtered-By: Mailman/MimeDel 2.1.29 Subject: Re: [FFmpeg-devel] [PATCH 3/5] avcodec/h264chroma_template: Replace variable by constant in chroma mc X-BeenThere: ffmpeg-devel@ffmpeg.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: FFmpeg development discussions and patches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , From: Kieran Kunhya via ffmpeg-devel Reply-To: FFmpeg development discussions and patches Cc: Kieran Kunhya Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" Archived-At: List-Archive: List-Post: On Sun, 13 Jul 2025, 02:11 Michael Niedermayer, wrote: > Signed-off-by: Michael Niedermayer > --- > libavcodec/h264chroma_template.c | 30 +++++++++++++++--------------- > 1 file changed, 15 insertions(+), 15 deletions(-) > > diff --git a/libavcodec/h264chroma_template.c > b/libavcodec/h264chroma_template.c > index b9d24f5a0cd..b20811a71ae 100644 > --- a/libavcodec/h264chroma_template.c > +++ b/libavcodec/h264chroma_template.c > @@ -54,7 +54,7 @@ static void FUNCC(OPNAME ## h264_chroma_mc1)(uint8_t > *_dst /*align 8*/, const ui > }\ > } else {\ > for(i=0; i - OP(dst[0], (A*src[0]));\ > + OP(dst[0], (64*src[0]));\ > dst+= stride;\ > src+= stride;\ > }\ > @@ -91,8 +91,8 @@ static void FUNCC(OPNAME ## h264_chroma_mc2)(uint8_t > *_dst /*align 8*/, const ui > }\ > } else {\ > for ( i = 0; i < h; i++){\ > - OP(dst[0], A * src[0]);\ > - OP(dst[1], A * src[1]);\ > + OP(dst[0], 64 * src[0]);\ > + OP(dst[1], 64 * src[1]);\ > dst += stride;\ > src += stride;\ > }\ > @@ -134,10 +134,10 @@ static void FUNCC(OPNAME ## h264_chroma_mc4)(uint8_t > *_dst /*align 8*/, const ui > }\ > } else {\ > for ( i = 0; i < h; i++){\ > - OP(dst[0], A * src[0]);\ > - OP(dst[1], A * src[1]);\ > - OP(dst[2], A * src[2]);\ > - OP(dst[3], A * src[3]);\ > + OP(dst[0], 64 * src[0]);\ > + OP(dst[1], 64 * src[1]);\ > + OP(dst[2], 64 * src[2]);\ > + OP(dst[3], 64 * src[3]);\ > dst += stride;\ > src += stride;\ > }\ > @@ -187,14 +187,14 @@ static void FUNCC(OPNAME ## h264_chroma_mc8)(uint8_t > *_dst /*align 8*/, const ui > }\ > } else {\ > for ( i = 0; i < h; i++){\ > - OP(dst[0], A * src[0]);\ > - OP(dst[1], A * src[1]);\ > - OP(dst[2], A * src[2]);\ > - OP(dst[3], A * src[3]);\ > - OP(dst[4], A * src[4]);\ > - OP(dst[5], A * src[5]);\ > - OP(dst[6], A * src[6]);\ > - OP(dst[7], A * src[7]);\ > + OP(dst[0], 64 * src[0]);\ > + OP(dst[1], 64 * src[1]);\ > + OP(dst[2], 64 * src[2]);\ > + OP(dst[3], 64 * src[3]);\ > + OP(dst[4], 64 * src[4]);\ > + OP(dst[5], 64 * src[5]);\ > + OP(dst[6], 64 * src[6]);\ > + OP(dst[7], 64 * src[7]);\ > dst += stride;\ > src += stride;\ > }\ > -- > 2.49.0 > Some explanation of what this is for would help. Kieran > _______________________________________________ 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".