i might just do the hash unconditionally in `visit_item`; e.g. a non-pub function may still get inlined in a downstream crate, and thus we should include it in the SVH, right?  (I suppose the answer to that question depends on what I can learn via the reachability pass; I still need to look at that.)