chore: refactor add_definition_location to take a Location

[asterite]

This method:

noir/compiler/noirc_frontend/src/locations.rs

Line 194 in 29d2d8a

pub(crate) fn add_definition_location(

looks up the location of a definition to register it. It's a bit redundant since when we are defining something we know its location. Calling this before the definition location is known could lead to bugs like this: https://github.com/noir-lang/noir/pull/7174/files#r1928537789