On Tue, May 09, 2023 at 11:25:03AM -0700, Evan Green wrote: > + if (riscv_isa_extension_available(isainfo->isa, ZBA)) > + pair->value |= RISCV_HWPROBE_EXT_ZBA; > + else > + missing |= RISCV_HWPROBE_EXT_ZBA; Part of me wonders if we should just have a macro for this, since it's just boilerplate w/ the only "variable" being the extension name. Just a thought, and we'll have plenty more cracks at changing things here, so: Reviewed-by: Conor Dooley <conor.dooley@xxxxxxxxxxxxx> Thanks, Conor.
Attachment:
signature.asc
Description: PGP signature
