On Thu Feb 8 08:22:34 2024 +0000, Nikolay Sivov wrote:
I see now why additional field was easier - 'size' is set later on, that involves extern_vars which is not populated initially, and that depends on uniform_copy() which, if separated, depends on entry point presence. Still, it's better to use the same field everywhere, I'll see how I can separate this in a way that makes sense. E.g. there is no need to handle function parameters for effects, I don't think it's possible to use them there, and have $Params block written out as a result.
I pushed something for this. It does introduce some duplication. To avoid it, _emit() has to change to set buffer "size" and update variable offsets without adding <temp> variables.