Every Target must have an address, correct? Does it sound like a performance issue for every

Field

instance to store the

Address

for its owning

Target

? This would allow substantially better error messages when validating fields, e.g. being able to say `"Invalid compatibility field for target //:lib`“` rather than

“Invalid compatibility field”

. We also need the

Address

for hydrating certain fields like

Sources

. But, storing

Address

on every field seems like a performance concern. Is it a premature optimization to worry about that performance concern? Is it worth it for the sake of substantially better error messages?