> On 20 Mar 2023, at 11:01, Christian Lindig <christian.lin...@cloud.com> wrote: > > > >> On 17 Mar 2023, at 13:19, Luca Fancellu <luca.fance...@arm.com> wrote: >> >> >> arch_cap_flags = caml_alloc_small(1, arch_cap_flags_tag); >> Store_field(arch_cap_flags, 0, arch_cap_list); >> Store_field(physinfo, 10, arch_cap_flags); >> +#elif defined(__aarch64__) >> + Store_field(physinfo, 10, Val_int(c_physinfo.arch_capabilities)); >> +#else >> + caml_failwith("Unhandled architecture"); >> +#endif >> > > Is this code overwriting an existing entry that was computed but now isn’t > used? If so, should the conditional compilation not avoid this?
Hi Christian, I don’t understand, what entry is being overwritten? If I understood it correctly, I’m writing the position 10 of physinfo which is not written before. Cheers, Luca > > — C