On 3/12/20 7:58 AM, LIU Zhiwei wrote: > +/* > + * vfwcvt.f.f.v vd, vs2, vm # > + * Convert single-width float to double-width float. > + */ > +static uint32_t vfwcvtffv16(uint16_t a, float_status *s) > +{ > + return float16_to_float32(a, true, s); > +} > +static uint64_t vfwcvtffv32(uint32_t a, float_status *s) > +{ > + return float32_to_float64(a, s); > +}
Do you actually need this second one, as opposed to using float32_to_float64 directly? > +RVVCALL(OPFVV1, vfwcvt_f_f_v_h, WOP_UU_H, H4, H2, vfwcvtffv16) > +RVVCALL(OPFVV1, vfwcvt_f_f_v_w, WOP_UU_W, H8, H4, vfwcvtffv32) > +GEN_VEXT_V_ENV(vfwcvt_f_f_v_h, 2, 4, clearl) > +GEN_VEXT_V_ENV(vfwcvt_f_f_v_w, 4, 8, clearq) > Otherwise, Reviewed-by: Richard Henderson <richard.hender...@linaro.org> r~