On 8/30/23 13:56, Philippe Mathieu-Daudé wrote:
+    for (unsigned i = 0, n = vdso->reloc_count; i < n; i++) {

Do we really need 'n'?

You should always have an loop invariant condition if possible.

vdso->reloc_count doesn't seem updated, but I get your point.

But the compiler doesn't know that -- it must assume that the store to *addr may overlap reloc_count.


r~

Reply via email to