On Mon, Jul 11, 2022, Yang Weijiang wrote: > - eax.full = id.a; > - ebx.full = id.b; > - edx.full = id.d; > + eax.full = pmu_arch_info(); > + ebx.full = pmu_gp_events(); > + edx.full = pmu_fixed_counters(); Adding helpers for individual fields but then caching the full fields and ignoring the helpers is silly. It doesn't require much more work to get rid of the unions entirely (see the pull request I sent to Paolo).