Re: RFR: 8352504: RISC-V: implement and enable CMoveI/L [v3]

2025-04-10 Thread Hamlin Li
On Wed, 9 Apr 2025 06:52:26 GMT, Fei Yang wrote: >> This is to not enable Zicond automatically, but user can still turn it on >> manually if they want to try or make sure it bring benefit on the specific >> hardware. >> Currently it's based on bananapi result, so maybe in the future we should

Re: RFR: 8352504: RISC-V: implement and enable CMoveI/L [v3]

2025-04-10 Thread Hamlin Li
> Hi, > Can you help to review this patch? > On riscv, CMoveI/L already were implemented, but there are some gap: > 1. CMoveI/L does not support comparison with float/double, corresponding > tests are not turn on either. > 2. Some optimization of C2 is not turned on, e.g. `Phi -> CMove -> min_max`