From patchwork Sat Jul 27 19:39:35 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: =?utf-8?q?R=C3=A9mi_Denis-Courmont?= X-Patchwork-Id: 50762 Delivered-To: ffmpegpatchwork2@gmail.com Received: by 2002:a05:612c:16f:b0:489:2eb3:e4c4 with SMTP id h47csp371801vqi; Sat, 27 Jul 2024 12:40:01 -0700 (PDT) X-Forwarded-Encrypted: i=2; AJvYcCWvSfA2j8zYxPtjPZ54khLOAgWbZAmsAJb4oCN7e2b+467SNtJbTShf8vv+Dq6sQ3Eqwq+diAPoGyWgz0VqJp3iisZypNnJkc+4HA== X-Google-Smtp-Source: AGHT+IEsBzpDdaslg38HQWaatnJkhazmm4amHJsvCKjfOJMeBRYeRVvTsz6q0j2MEzaCHxSiMZ7y X-Received: by 2002:a2e:b178:0:b0:2ef:2593:334d with SMTP id 38308e7fff4ca-2f12ee579aemr19162181fa.47.1722109200798; Sat, 27 Jul 2024 12:40:00 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1722109200; cv=none; d=google.com; s=arc-20160816; b=cGcZOJNb+5gIIgWmOfH/CFXWCQOCJvyFKxZCIAEuXlXwL43cbPhW+iiKZXkBY8JFQx ddCpsjebGsrIojMsALT5Ns1BWf798ClnQZGyYQKbRbP0uxS+BDYgkaiAPe8zNT25uAWs NqNAywKOqza1rQG6bhv4t+dbcd9ozKMwAxA3+JtOrIkGkp4vqEIl36w/8Nv7FaIrZHGE Csj0D6y4fE2LZS/oOGtpH/BXN2HS0+XI+o56iekpziwGCHwzd8vW97Cr/25tw1UgCJF5 dKf8XBPaEIUOQoqh4APGUaMraTi+mY3k6rKETcPN7T4VA/j1nC3Vl3ANOI9eCDS6ukKN yrig== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=sender:errors-to:content-transfer-encoding:reply-to:list-subscribe :list-help:list-post:list-archive:list-unsubscribe:list-id :precedence:subject:mime-version:references:in-reply-to:message-id :date:to:from:delivered-to; bh=pjgWn7eCdJYhihxDLAzwdF0r4p+rVTlhgBzc2ZZ7r/E=; fh=YOA8vD9MJZuwZ71F/05pj6KdCjf6jQRmzLS+CATXUQk=; b=YTufgIr1TscEYk0qltD/3M8IOuVv5qyLpNFL1IbI829izfhPxOZsKdJp+eWt6JImuP pFmy6OahnAl12dfaELCXuPxV6dX6JHtbkInYxfS6VEds1wa4w83064fGUlG3de7o+Mx7 JNwhoSP5TCJPkJFHMX5u0mY0pIvSnFXKD88Ju/dRuaqVskGNIIT3uWm5uMeoxhUp5Nmk Fx7X1QFN8DLZv3gHBwjzOTlR1xIAH3pIC8nRMwoxOjuBmJRBI3b1AzeEtXIz0ye3nnjx LlsBVbhv1u4XPgm1gyFcG6m/0WuyqnOwrF/ZloIlrQHk28xlGkxzK4rJAAkrkB4SLK05 vduQ==; dara=google.com ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of ffmpeg-devel-bounces@ffmpeg.org designates 79.124.17.100 as permitted sender) smtp.mailfrom=ffmpeg-devel-bounces@ffmpeg.org Return-Path: Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org. [79.124.17.100]) by mx.google.com with ESMTP id 38308e7fff4ca-2f03d13eadesi16244511fa.205.2024.07.27.12.40.00; Sat, 27 Jul 2024 12:40:00 -0700 (PDT) Received-SPF: pass (google.com: domain of ffmpeg-devel-bounces@ffmpeg.org designates 79.124.17.100 as permitted sender) client-ip=79.124.17.100; Authentication-Results: mx.google.com; spf=pass (google.com: domain of ffmpeg-devel-bounces@ffmpeg.org designates 79.124.17.100 as permitted sender) smtp.mailfrom=ffmpeg-devel-bounces@ffmpeg.org Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 9F06768CC49; Sat, 27 Jul 2024 22:39:44 +0300 (EEST) X-Original-To: ffmpeg-devel@ffmpeg.org Delivered-To: ffmpeg-devel@ffmpeg.org Received: from ursule.remlab.net (vps-a2bccee9.vps.ovh.net [51.75.19.47]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id B70FC68D506 for ; Sat, 27 Jul 2024 22:39:36 +0300 (EEST) Received: from basile.remlab.net (localhost [IPv6:::1]) by ursule.remlab.net (Postfix) with ESMTP id 35405C01A0 for ; Sat, 27 Jul 2024 22:39:36 +0300 (EEST) From: =?utf-8?q?R=C3=A9mi_Denis-Courmont?= To: ffmpeg-devel@ffmpeg.org Date: Sat, 27 Jul 2024 22:39:35 +0300 Message-ID: <20240727193935.85724-2-remi@remlab.net> X-Mailer: git-send-email 2.45.2 In-Reply-To: <20240727193935.85724-1-remi@remlab.net> References: <20240727193935.85724-1-remi@remlab.net> MIME-Version: 1.0 Subject: [FFmpeg-devel] [PATCH 2/2] lavc/vc1dsp: use saturating arithmetic for RVV inv_trans_dc 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: , Reply-To: FFmpeg development discussions and patches Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" X-TUID: MdabgIPSm7gt T-Head C908 (cycles): vc1dsp.vc1_inv_trans_4x4_dc_c: 113.7 vc1dsp.vc1_inv_trans_4x4_dc_rvv_i32: 46.5 (before) vc1dsp.vc1_inv_trans_4x4_dc_rvv_i32: 45.5 (after) vc1dsp.vc1_inv_trans_4x8_dc_c: 230.7 vc1dsp.vc1_inv_trans_4x8_dc_rvv_i32: 65.7 (before) vc1dsp.vc1_inv_trans_4x8_dc_rvv_i32: 52.5 (after) vc1dsp.vc1_inv_trans_8x4_dc_c: 246.7 vc1dsp.vc1_inv_trans_8x4_dc_rvv_i64: 56.7 (before) vc1dsp.vc1_inv_trans_8x4_dc_rvv_i64: 45.5 (after) vc1dsp.vc1_inv_trans_8x8_dc_c: 419.7 vc1dsp.vc1_inv_trans_8x8_dc_rvv_i64: 81.2 (before) vc1dsp.vc1_inv_trans_8x8_dc_rvv_i64: 53.5 (after) --- libavcodec/riscv/vc1dsp_rvv.S | 22 ++++++++++++---------- 1 file changed, 12 insertions(+), 10 deletions(-) diff --git a/libavcodec/riscv/vc1dsp_rvv.S b/libavcodec/riscv/vc1dsp_rvv.S index 548ef9d3bf..f9b59688ae 100644 --- a/libavcodec/riscv/vc1dsp_rvv.S +++ b/libavcodec/riscv/vc1dsp_rvv.S @@ -38,18 +38,20 @@ func ff_vc1_inv_trans_\cols\()x\rows\()_dc_rvv, zve64x, zba .endif addi t2, t2, 64 srai t2, t2, 7 -.if \rows * \cols == 64 - vsetvli zero, t0, e16, m8, ta, ma -.elseif \rows * \cols == 32 - vsetvli zero, t0, e16, m4, ta, ma +.if \rows * \cols >= 32 + vsetvli zero, t0, e8, m\mat_lmul, ta, ma .else - vsetivli zero, \rows * \cols, e16, m2, ta, ma + vsetivli zero, \rows * \cols, e8, m\mat_lmul, ta, ma .endif - vzext.vf2 v8, v0 - vadd.vx v8, v8, t2 - vmax.vx v8, v8, zero - vsetvli zero, zero, e8, m\mat_lmul, ta, ma - vnclipu.wi v0, v8, 0 + bgez t2, 1f + + neg t2, t2 + vssubu.vx v0, v0, t2 + vsetivli zero, \rows, e8, m\row_lmul, ta, ma + vsse\w\().v v0, (a0), a1 + ret +1: + vsaddu.vx v0, v0, t2 vsetivli zero, \rows, e8, m\row_lmul, ta, ma vsse\w\().v v0, (a0), a1 ret