Should classify_name take Source<ast::Name> instead of FileId, &ast::Name? I'm working on getting hover working through macros like goto def now does.
classify_name
Source<ast::Name>
FileId, &ast::Name
@Jeremy Kolb yep!
Cool. Filed it as a separate PR