Re: [PATCH] Findirect_function

[Manuel Giraud]

Mattias Engdegård <mattiase@acm.org> writes:

> 14 jan. 2022 kl. 11.40 skrev Manuel Giraud <manuel@ledu-giraud.fr>:
>
>> Maybe it is bike shedding territory (and maybe I'm missing something)
>
> Not at all, patches are always welcome!

Thanks.

[...]

> Good! See if you can find more instances of this unfortunate
> pattern. Even if it makes no difference for the generated code (very
> likely), it obfuscates the source code.

Ok, I'll look for this kind of pattern.

> The other changes are less obvious since they may introduce extra
> costs; some of these code paths are fairly hot. In addition, I'm
> giving part of the interpreter an overhaul so perhaps we could drop
> them for the time being. Do you agree?

Yes it is one more C function call. Sure you could drop it!
-- 
Manuel Giraud