> I'd worry there's other latent bugs of this nature and if we'd be better
> off avoiding the temporary sharing.  We have structure sharing rules for
> a reason -- I'd hate to think of all the code that would need auditing
> to ensure it was safe with this bogus sharing.

I wouldn't throw the baby with the bath's water here, it's one of the numerous 
PRs opened by Zhendong Su and which clearly look machine-generated.  It fails 
with 4.4.x onwards and apparently nobody noticed the problem in real life.
We know that we have latent sharing issues in the combiner because of the way 
it's designed, but they are quite rare in practice.

-- 
Eric Botcazou

Reply via email to