You probably won’t need to add much to resolve-names.cc, so a new BlockDataVisitor class seems unnecessary. I suggest starting by adding to ResolveNamesVisitor. If what you add turns out to be similar to what is in SubprogramVisitor then that may be a good place to move it.
Tim