You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
In #6811, we relaxed the owning module constraint for local targets. This should be safe, but is generally not something we want to do.
The problem is we have local targets to instances that have been extracted beyond the owning module boundary intentionally by ExtractInstances, but we still want to target them within the component they were originally a part of.
The change allows us to target them, and is safe, since it still requires a single instantiation site, but this extraction doesn't play well with the encapsulation of a component in general. Once we get off that approach, we should revert the above PR to restore the original constraint.
The text was updated successfully, but these errors were encountered:
In #6811, we relaxed the owning module constraint for local targets. This should be safe, but is generally not something we want to do.
The problem is we have local targets to instances that have been extracted beyond the owning module boundary intentionally by ExtractInstances, but we still want to target them within the component they were originally a part of.
The change allows us to target them, and is safe, since it still requires a single instantiation site, but this extraction doesn't play well with the encapsulation of a component in general. Once we get off that approach, we should revert the above PR to restore the original constraint.
The text was updated successfully, but these errors were encountered: