On 12/22/2013 02:49 PM, Peter Maydell wrote:
> define_one_arm_cp_reg_with_opaque() has a set of nested loops which
> insert a cpreg entry into the hashtable for each of the possible
> opc/crn/crm values allowed by wildcard specifications. We're about
> to add an extra loop to this nesting, so pull the core of the loop
> (which adds a single entry to the hashtable) out into its own
> function for clarity.
> 
> Signed-off-by: Peter Maydell <peter.mayd...@linaro.org>
> ---
>  target-arm/helper.c | 94 
> +++++++++++++++++++++++++++++------------------------
>  1 file changed, 52 insertions(+), 42 deletions(-)

Reviewed-by: Richard Henderson <r...@twiddle.net>


r~

Reply via email to