Re: [PATCH v4 12/45] target/arm: Merge allocation of the cpreg and its name

2022-05-03 Thread Peter Maydell
On Sun, 1 May 2022 at 07:18, Richard Henderson wrote: > > Simplify freeing cp_regs hash table entries by using a single > allocation for the entire value. > > This fixes a theoretical bug if we were to ever free the entire > hash table, because we've been installing string literal constants > into

[PATCH v4 12/45] target/arm: Merge allocation of the cpreg and its name

2022-04-30 Thread Richard Henderson
Simplify freeing cp_regs hash table entries by using a single allocation for the entire value. This fixes a theoretical bug if we were to ever free the entire hash table, because we've been installing string literal constants into the cpreg structure in define_arm_vh_e2h_redirects_aliases. However