On 11/19/20 3:55 PM, Peter Maydell wrote: > +/* > + * Emit code to store the sysreg to its final destination; frees the > + * TCG temp 'value' it is passed. > + */ > +typedef void fp_sysreg_storefn(DisasContext *s, void *opaque, TCGv_i32 > value); > +/* > + * Emit code to load the value to be copied to the sysreg; returns > + * a new TCG temporary > + */ > +typedef TCGv_i32 fp_sysreg_loadfn(DisasContext *s, void *opaque); > + > +/* Common decode/access checks for fp sysreg read/write */ > +typedef enum fp_sysreg_check_result { > + fp_sysreg_check_failed, /* caller should return false */ > + fp_sysreg_check_done, /* caller should return true */ > + fp_sysreg_check_continue, /* caller should continue generating code */ > +} fp_sysreg_check_result;
CamelCase? Otherwise, Reviewed-by: Richard Henderson <richard.hender...@linaro.org> r~