refactor: change the way of checking variable has been declared

[zhangpanweb]

写了一个对比的 visitor，让所有的 module 都跑 这个 visitor， visitor有两个版本，一个用 collect_decls，一个用 unresolved_mark 判定。用 normal的 example 来跑 puffin（yuyanAssets 跑了会栈溢出， examples/normal 已能看出差异） 。如下

collect_decls版本：

struct DiffCollectDecls {
    bindings: Lrc<AHashSet<Id>>,
}

impl VisitMut for DiffCollectDecls {
    fn visit_mut_module(&mut self, module: &mut Module) {
        mako_profile_scope!("node_stuff_visit_module");

        self.bindings = Lrc::new(collect_decls(&*module));
        module.visit_mut_children_with(self);
    }
}

puffin stat 如下

unresolved_mark版本

struct DiffCollectDecls {
    unresolved_mark: Mark,
}

impl VisitMut for DiffCollectDecls {
    fn visit_mut_module(&mut self, module: &mut Module) {
        mako_profile_scope!("node_stuff_visit_module");

        if module.span.ctxt.outer() == self.unresolved_mark {}
        module.visit_mut_children_with(self);
    }
}

puffin stat 如下

结论：unresolved_mark 有明显优势 ，pr 把代码中通过 collect_decls bindings 的判定方式都改为使用unresolved_mark的方式