[PATCH] internal-fn: Fix bug of BIAS argument index

[PATCH] internal-fn: Fix bug of BIAS argument index

[juzhe.zhong]

From: Ju-Zhe Zhong <juzhe.zhong@rivai.ai>

When trying to enable LEN_MASK_{LOAD,STORE} in RISC-V port,
I found I made a mistake in case of argument index of BIAS.

This patch is an obvious fix,

Ok for trunk ?

gcc/ChangeLog:

        * internal-fn.cc (expand_partial_store_optab_fn): Fix bug of BIAS argument index.

---
 gcc/internal-fn.cc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/gcc/internal-fn.cc b/gcc/internal-fn.cc
index 1c2fd487e2a..9017176dc7a 100644
--- a/gcc/internal-fn.cc
+++ b/gcc/internal-fn.cc
@@ -2991,7 +2991,7 @@ expand_partial_store_optab_fn (internal_fn ifn, gcall *stmt, convert_optab optab
       maskt = gimple_call_arg (stmt, 3);
       mask = expand_normal (maskt);
       create_input_operand (&ops[3], mask, TYPE_MODE (TREE_TYPE (maskt)));
-      biast = gimple_call_arg (stmt, 4);
+      biast = gimple_call_arg (stmt, 5);
       bias = expand_normal (biast);
       create_input_operand (&ops[4], bias, QImode);
       icode = convert_optab_handler (optab, TYPE_MODE (type), GET_MODE (mask));
-- 
2.36.3

Re: [PATCH] internal-fn: Fix bug of BIAS argument index

[Jeff Law]

On 6/24/23 21:36, juzhe.zhong@rivai.ai wrote:
> From: Ju-Zhe Zhong <juzhe.zhong@rivai.ai>
> 
> When trying to enable LEN_MASK_{LOAD,STORE} in RISC-V port,
> I found I made a mistake in case of argument index of BIAS.
> 
> This patch is an obvious fix,
> 
> Ok for trunk ?
> 
> gcc/ChangeLog:
> 
>          * internal-fn.cc (expand_partial_store_optab_fn): Fix bug of BIAS argument index.
OK
jeff

RE: [PATCH] internal-fn: Fix bug of BIAS argument index

[Li, Pan2]

Committed, thanks Jeff.

Pan

-----Original Message-----
From: Gcc-patches <gcc-patches-bounces+pan2.li=intel.com@gcc.gnu.org> On Behalf Of Jeff Law via Gcc-patches
Sent: Sunday, June 25, 2023 8:48 PM
To: juzhe.zhong@rivai.ai; gcc-patches@gcc.gnu.org
Cc: richard.sandiford@arm.com; rguenther@suse.de
Subject: Re: [PATCH] internal-fn: Fix bug of BIAS argument index



On 6/24/23 21:36, juzhe.zhong@rivai.ai wrote:
> From: Ju-Zhe Zhong <juzhe.zhong@rivai.ai>
> 
> When trying to enable LEN_MASK_{LOAD,STORE} in RISC-V port,
> I found I made a mistake in case of argument index of BIAS.
> 
> This patch is an obvious fix,
> 
> Ok for trunk ?
> 
> gcc/ChangeLog:
> 
>          * internal-fn.cc (expand_partial_store_optab_fn): Fix bug of BIAS argument index.
OK
jeff