> Please still remove the leading '*' of the comment.  The series is OK with 
> that 
> fixed.  Thanks for your patience.

Thanks Robin for help and explanation. I will commit with these changes, it 
looks like auto inserted when enter.

> We can take care of a more comprehensive rtx_cost once all the follow ups are 
> in.

Sure thing.

Pan

-----Original Message-----
From: Robin Dapp <rdapp....@gmail.com> 
Sent: Tuesday, May 6, 2025 8:17 PM
To: Li, Pan2 <pan2...@intel.com>; gcc-patches@gcc.gnu.org
Cc: juzhe.zh...@rivai.ai; kito.ch...@gmail.com; jeffreya...@gmail.com; 
rdapp....@gmail.com; Chen, Ken <ken.c...@intel.com>; Liu, Hongtao 
<hongtao....@intel.com>; Robin Dapp <rdapp....@gmail.com>
Subject: Re: [PATCH v4 2/6] RISC-V: Add gr2vr cost helper function

> +/*
> + * Return the cost of operation that move from gpr to vr.
> + *
> + * It will take the value of --param=gpr2vr_cost if it is provided.
> + * Or the default regmove->GR2VR will be returned.
> + */

Please still remove the leading '*' of the comment.  The series is OK with that 
fixed.  Thanks for your patience.

We can take care of a more comprehensive rtx_cost once all the follow ups are 
in.

-- 
Regards
 Robin

Reply via email to