> 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


Reply via email to