> 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 <[email protected]> 
Sent: Tuesday, May 6, 2025 8:17 PM
To: Li, Pan2 <[email protected]>; [email protected]
Cc: [email protected]; [email protected]; [email protected]; 
[email protected]; Chen, Ken <[email protected]>; Liu, Hongtao 
<[email protected]>; Robin Dapp <[email protected]>
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