stop using csearch in librustc and librustc_lint

Author: arielb1

src/librustc/metadata/csearch.rs

@@ -406,4 +406,3 @@ pub fn def_path(tcx: &ty::ctxt, def: DefId) -> ast_map::DefPath {
     let local_path = cdata.local_def_path();
     local_path.into_iter().chain(path).collect()
 }
-

src/librustc/metadata/mod.rs

@@ -21,3 +21,4 @@ pub mod loader;
 pub mod filesearch;
 pub mod macro_import;
 pub mod inline;
+pub mod util;

src/librustc/metadata/util.rs

@@ -0,0 +1,324 @@
+// Copyright 2015 The Rust Project Developers. See the COPYRIGHT
+// file at the top-level directory of this distribution and at
+// http://rust-lang.org/COPYRIGHT.
+//
+// Licensed under the Apache License, Version 2.0 <LICENSE-APACHE or
+// http://www.apache.org/licenses/LICENSE-2.0> or the MIT license
+// <LICENSE-MIT or http://opensource.org/licenses/MIT>, at your
+// option. This file may not be copied, modified, or distributed
+// except according to those terms.
+
+use front::map as ast_map;
+use metadata::cstore;
+use metadata::decoder;
+use middle::astencode;
+use middle::lang_items;
+use middle::ty;
+use middle::def_id::{DefId, DefIndex};
+
+use std::rc::Rc;
+use syntax::ast;
+use syntax::attr;
+use rustc_front::hir;
+
+pub use metadata::csearch::FoundAst;
+pub use metadata::cstore::LinkagePreference;
+pub use metadata::decoder::DecodeInlinedItem;
+pub use metadata::inline::InlinedItem;
+
+pub trait CrateStore<'tcx> {
+    // item info
+    fn stability(&self, def: DefId) -> Option<attr::Stability>;
+    fn closure_kind(&self, tcx: &ty::ctxt<'tcx>, def_id: DefId)
+                    -> ty::ClosureKind;
+    fn closure_ty(&self, tcx: &ty::ctxt<'tcx>, def_id: DefId)
+                  -> ty::ClosureTy<'tcx>;
+    fn item_variances(&self, def: DefId) -> ty::ItemVariances;
+    fn repr_attrs(&self, def: DefId) -> Vec<attr::ReprAttr>;
+    fn item_type(&self, tcx: &ty::ctxt<'tcx>, def: DefId)
+                 -> ty::TypeScheme<'tcx>;
+    fn item_path(&self, def: DefId) -> Vec<ast_map::PathElem>;
+    fn item_name(&self, def: DefId) -> ast::Name;
+    fn item_predicates(&self, tcx: &ty::ctxt<'tcx>, def: DefId)
+                       -> ty::GenericPredicates<'tcx>;
+    fn item_super_predicates(&self, tcx: &ty::ctxt<'tcx>, def: DefId)
+                             -> ty::GenericPredicates<'tcx>;
+    fn item_attrs(&self, def_id: DefId) -> Vec<ast::Attribute>;
+    fn trait_def(&self, tcx: &ty::ctxt<'tcx>, def: DefId)-> ty::TraitDef<'tcx>;
+    fn adt_def(&self, tcx: &ty::ctxt<'tcx>, def: DefId) -> ty::AdtDefMaster<'tcx>;
+    fn inherent_implementations_for_type(&self, def_id: DefId) -> Vec<DefId>;
+
+    // trait info
+    fn implementations_of_trait(&self, def_id: DefId) -> Vec<DefId>;
+    fn provided_trait_methods(&self, tcx: &ty::ctxt<'tcx>, def: DefId)
+                              -> Vec<Rc<ty::Method<'tcx>>>;
+    fn trait_item_def_ids(&self, def: DefId)
+                          -> Vec<ty::ImplOrTraitItemId>;
+
+    // impl info
+    fn impl_items(&self, impl_def_id: DefId) -> Vec<ty::ImplOrTraitItemId>;
+    fn impl_trait_ref(&self, tcx: &ty::ctxt<'tcx>, def: DefId)
+                      -> Option<ty::TraitRef<'tcx>>;
+    fn impl_polarity(&self, def: DefId) -> Option<hir::ImplPolarity>;
+    fn custom_coerce_unsized_kind(&self, def: DefId)
+                                  -> Option<ty::adjustment::CustomCoerceUnsized>;
+    fn associated_consts(&self, tcx: &ty::ctxt<'tcx>, def: DefId)
+                         -> Vec<Rc<ty::AssociatedConst<'tcx>>>;
+
+    // trait/impl-item info
+    fn trait_of_item(&self, tcx: &ty::ctxt<'tcx>, def_id: DefId)
+                     -> Option<DefId>;
+    fn impl_or_trait_item(&self, tcx: &ty::ctxt<'tcx>, def: DefId)
+                          -> ty::ImplOrTraitItem<'tcx>;
+
+    // flags
+    fn is_const_fn(&self, did: DefId) -> bool;
+    fn is_defaulted_trait(&self, did: DefId) -> bool;
+    fn is_impl(&self, did: DefId) -> bool;
+
+    // metadata
+    fn dylib_dependency_formats(&self, cnum: ast::CrateNum)
+                                    -> Vec<(ast::CrateNum, cstore::LinkagePreference)>;
+    fn lang_items(&self, cnum: ast::CrateNum) -> Vec<(DefIndex, usize)>;
+    fn missing_lang_items(&self, cnum: ast::CrateNum)
+                          -> Vec<lang_items::LangItem>;
+    fn is_staged_api(&self, cnum: ast::CrateNum) -> bool;
+
+    // misc.
+    fn def_path(&self, def: DefId) -> ast_map::DefPath;
+    fn maybe_get_item_ast(&'tcx self, tcx: &ty::ctxt<'tcx>, def: DefId)
+                          -> FoundAst<'tcx>;
+}
+
+impl<'tcx> CrateStore<'tcx> for cstore::CStore {
+    fn stability(&self, def: DefId) -> Option<attr::Stability>
+    {
+        let cdata = self.get_crate_data(def.krate);
+        decoder::get_stability(&*cdata, def.index)
+    }
+
+    fn closure_kind(&self, _tcx: &ty::ctxt<'tcx>, def_id: DefId) -> ty::ClosureKind
+    {
+        assert!(!def_id.is_local());
+        let cdata = self.get_crate_data(def_id.krate);
+        decoder::closure_kind(&*cdata, def_id.index)
+    }
+
+    fn closure_ty(&self, tcx: &ty::ctxt<'tcx>, def_id: DefId) -> ty::ClosureTy<'tcx>
+    {
+        assert!(!def_id.is_local());
+        let cdata = self.get_crate_data(def_id.krate);
+        decoder::closure_ty(&*cdata, def_id.index, tcx)
+    }
+
+    fn item_variances(&self, def: DefId) -> ty::ItemVariances {
+        let cdata = self.get_crate_data(def.krate);
+        decoder::get_item_variances(&*cdata, def.index)
+    }
+
+    fn repr_attrs(&self, def: DefId) -> Vec<attr::ReprAttr> {
+        let cdata = self.get_crate_data(def.krate);
+        decoder::get_repr_attrs(&*cdata, def.index)
+    }
+
+    fn item_type(&self, tcx: &ty::ctxt<'tcx>, def: DefId)
+                 -> ty::TypeScheme<'tcx>
+    {
+        let cdata = self.get_crate_data(def.krate);
+        decoder::get_type(&*cdata, def.index, tcx)
+    }
+
+    fn item_predicates(&self, tcx: &ty::ctxt<'tcx>, def: DefId)
+                       -> ty::GenericPredicates<'tcx>
+    {
+        let cdata = self.get_crate_data(def.krate);
+        decoder::get_predicates(&*cdata, def.index, tcx)
+    }
+
+    fn item_super_predicates(&self, tcx: &ty::ctxt<'tcx>, def: DefId)
+                             -> ty::GenericPredicates<'tcx>
+    {
+        let cdata = self.get_crate_data(def.krate);
+        decoder::get_super_predicates(&*cdata, def.index, tcx)
+    }
+
+    fn item_attrs(&self, def_id: DefId) -> Vec<ast::Attribute>
+    {
+        let cdata = self.get_crate_data(def_id.krate);
+        decoder::get_item_attrs(&*cdata, def_id.index)
+    }
+
+    fn trait_def(&self, tcx: &ty::ctxt<'tcx>, def: DefId) -> ty::TraitDef<'tcx>
+    {
+        let cdata = self.get_crate_data(def.krate);
+        decoder::get_trait_def(&*cdata, def.index, tcx)
+    }
+
+    fn adt_def(&self, tcx: &ty::ctxt<'tcx>, def: DefId) -> ty::AdtDefMaster<'tcx>
+    {
+        let cdata = self.get_crate_data(def.krate);
+        decoder::get_adt_def(&self.intr, &*cdata, def.index, tcx)
+    }
+
+    fn item_path(&self, def: DefId) -> Vec<ast_map::PathElem> {
+        let cdata = self.get_crate_data(def.krate);
+        let path = decoder::get_item_path(&*cdata, def.index);
+
+        cdata.with_local_path(|cpath| {
+            let mut r = Vec::with_capacity(cpath.len() + path.len());
+            r.push_all(cpath);
+            r.push_all(&path);
+            r
+        })
+    }
+
+    fn item_name(&self, def: DefId) -> ast::Name {
+        let cdata = self.get_crate_data(def.krate);
+        decoder::get_item_name(&self.intr, &cdata, def.index)
+    }
+
+
+    fn inherent_implementations_for_type(&self, def_id: DefId) -> Vec<DefId>
+    {
+        let mut result = vec![];
+        let cdata = self.get_crate_data(def_id.krate);
+        decoder::each_inherent_implementation_for_type(&*cdata, def_id.index,
+                                                       |iid| result.push(iid));
+        result
+    }
+
+    fn implementations_of_trait(&self, def_id: DefId) -> Vec<DefId>
+    {
+        let mut result = vec![];
+        self.iter_crate_data(|_, cdata| {
+            decoder::each_implementation_for_trait(cdata, def_id, &mut |iid| {
+                result.push(iid)
+            })
+        });
+        result
+    }
+
+    fn provided_trait_methods(&self, tcx: &ty::ctxt<'tcx>, def: DefId)
+                              -> Vec<Rc<ty::Method<'tcx>>>
+    {
+        let cdata = self.get_crate_data(def.krate);
+        decoder::get_provided_trait_methods(self.intr.clone(), &*cdata, def.index, tcx)
+    }
+
+    fn trait_item_def_ids(&self, def: DefId)
+                          -> Vec<ty::ImplOrTraitItemId>
+    {
+        let cdata = self.get_crate_data(def.krate);
+        decoder::get_trait_item_def_ids(&*cdata, def.index)
+    }
+
+    fn impl_items(&self, impl_def_id: DefId) -> Vec<ty::ImplOrTraitItemId>
+    {
+        let cdata = self.get_crate_data(impl_def_id.krate);
+        decoder::get_impl_items(&*cdata, impl_def_id.index)
+    }
+
+    fn impl_polarity(&self, def: DefId) -> Option<hir::ImplPolarity>
+    {
+        let cdata = self.get_crate_data(def.krate);
+        decoder::get_impl_polarity(&*cdata, def.index)
+    }
+
+    fn impl_trait_ref(&self, tcx: &ty::ctxt<'tcx>, def: DefId)
+                      -> Option<ty::TraitRef<'tcx>>
+    {
+        let cdata = self.get_crate_data(def.krate);
+        decoder::get_impl_trait(&*cdata, def.index, tcx)
+    }
+
+    fn custom_coerce_unsized_kind(&self, def: DefId)
+                                  -> Option<ty::adjustment::CustomCoerceUnsized>
+    {
+        let cdata = self.get_crate_data(def.krate);
+        decoder::get_custom_coerce_unsized_kind(&*cdata, def.index)
+    }
+
+    // FIXME: killme
+    fn associated_consts(&self, tcx: &ty::ctxt<'tcx>, def: DefId)
+                         -> Vec<Rc<ty::AssociatedConst<'tcx>>> {
+        let cdata = self.get_crate_data(def.krate);
+        decoder::get_associated_consts(self.intr.clone(), &*cdata, def.index, tcx)
+    }
+
+    fn trait_of_item(&self, tcx: &ty::ctxt<'tcx>, def_id: DefId) -> Option<DefId>
+    {
+        let cdata = self.get_crate_data(def_id.krate);
+        decoder::get_trait_of_item(&*cdata, def_id.index, tcx)
+    }
+
+    fn impl_or_trait_item(&self, tcx: &ty::ctxt<'tcx>, def: DefId)
+                          -> ty::ImplOrTraitItem<'tcx>
+    {
+        let cdata = self.get_crate_data(def.krate);
+        decoder::get_impl_or_trait_item(
+            self.intr.clone(),
+            &*cdata,
+            def.index,
+            tcx)
+    }
+
+    fn is_const_fn(&self, did: DefId) -> bool
+    {
+        let cdata = self.get_crate_data(did.krate);
+        decoder::is_const_fn(&cdata, did.index)
+    }
+
+    fn is_defaulted_trait(&self, trait_def_id: DefId) -> bool {
+        let cdata = self.get_crate_data(trait_def_id.krate);
+        decoder::is_defaulted_trait(&*cdata, trait_def_id.index)
+    }
+
+    fn is_impl(&self, did: DefId) -> bool {
+        let cdata = self.get_crate_data(did.krate);
+        decoder::is_impl(&*cdata, did.index)
+    }
+
+    fn dylib_dependency_formats(&self, cnum: ast::CrateNum)
+                                -> Vec<(ast::CrateNum, cstore::LinkagePreference)>
+    {
+        let cdata = self.get_crate_data(cnum);
+        decoder::get_dylib_dependency_formats(&cdata)
+    }
+
+    fn lang_items(&self, cnum: ast::CrateNum) -> Vec<(DefIndex, usize)>
+    {
+        let mut result = vec![];
+        let crate_data = self.get_crate_data(cnum);
+        decoder::each_lang_item(&*crate_data, |did, lid| {
+            result.push((did, lid)); true
+        });
+        result
+    }
+
+    fn missing_lang_items(&self, cnum: ast::CrateNum)
+                          -> Vec<lang_items::LangItem>
+    {
+        let cdata = self.get_crate_data(cnum);
+        decoder::get_missing_lang_items(&*cdata)
+    }
+
+    fn is_staged_api(&self, cnum: ast::CrateNum) -> bool
+    {
+        self.get_crate_data(cnum).staged_api
+    }
+
+    fn def_path(&self, def: DefId) -> ast_map::DefPath {
+        let cdata = self.get_crate_data(def.krate);
+        let path = decoder::def_path(&*cdata, def.index);
+        let local_path = cdata.local_def_path();
+        local_path.into_iter().chain(path).collect()
+    }
+
+    fn maybe_get_item_ast(&'tcx self, tcx: &ty::ctxt<'tcx>, def: DefId)
+                          -> FoundAst<'tcx>
+    {
+        let cdata = self.get_crate_data(def.krate);
+        let decode_inlined_item = Box::new(astencode::decode_inlined_item);
+        decoder::maybe_get_item_ast(&*cdata, tcx, def.index, decode_inlined_item)
+    }
+}

src/librustc/middle/const_eval.rs

@@ -16,9 +16,8 @@ use self::EvalHint::*;
 
 use front::map as ast_map;
 use front::map::blocks::FnLikeNode;
-use metadata::csearch;
-use metadata::inline::InlinedItem;
-use middle::{astencode, def, infer, subst, traits};
+use metadata::util::{self as mdutil, CrateStore, InlinedItem};
+use middle::{def, infer, subst, traits};
 use middle::def_id::DefId;
 use middle::pat_util::def_to_path;
 use middle::ty::{self, Ty};
@@ -145,13 +144,12 @@ pub fn lookup_const_by_id<'a, 'tcx: 'a>(tcx: &'a ty::ctxt<'tcx>,
             None => {}
         }
         let mut used_ref_id = false;
-        let expr_id = match csearch::maybe_get_item_ast(tcx, def_id,
-            Box::new(astencode::decode_inlined_item)) {
-            csearch::FoundAst::Found(&InlinedItem::Item(ref item)) => match item.node {
+        let expr_id = match tcx.sess.cstore.maybe_get_item_ast(tcx, def_id) {
+            mdutil::FoundAst::Found(&InlinedItem::Item(ref item)) => match item.node {
                 hir::ItemConst(_, ref const_expr) => Some(const_expr.id),
                 _ => None
             },
-            csearch::FoundAst::Found(&InlinedItem::TraitItem(trait_id, ref ti)) => match ti.node {
+            mdutil::FoundAst::Found(&InlinedItem::TraitItem(trait_id, ref ti)) => match ti.node {
                 hir::ConstTraitItem(_, _) => {
                     used_ref_id = true;
                     match maybe_ref_id {
@@ -170,7 +168,7 @@ pub fn lookup_const_by_id<'a, 'tcx: 'a>(tcx: &'a ty::ctxt<'tcx>,
                 }
                 _ => None
             },
-            csearch::FoundAst::Found(&InlinedItem::ImplItem(_, ref ii)) => match ii.node {
+            mdutil::FoundAst::Found(&InlinedItem::ImplItem(_, ref ii)) => match ii.node {
                 hir::ImplItemKind::Const(_, ref expr) => Some(expr.id),
                 _ => None
             },
@@ -196,15 +194,14 @@ fn inline_const_fn_from_external_crate(tcx: &ty::ctxt, def_id: DefId)
         None => {}
     }
 
-    if !csearch::is_const_fn(&tcx.sess.cstore, def_id) {
+    if !tcx.sess.cstore.is_const_fn(def_id) {
         tcx.extern_const_fns.borrow_mut().insert(def_id, ast::DUMMY_NODE_ID);
         return None;
     }
 
-    let fn_id = match csearch::maybe_get_item_ast(tcx, def_id,
-        box astencode::decode_inlined_item) {
-        csearch::FoundAst::Found(&InlinedItem::Item(ref item)) => Some(item.id),
-        csearch::FoundAst::Found(&InlinedItem::ImplItem(_, ref item)) => Some(item.id),
+    let fn_id = match tcx.sess.cstore.maybe_get_item_ast(tcx, def_id) {
+        mdutil::FoundAst::Found(&InlinedItem::Item(ref item)) => Some(item.id),
+        mdutil::FoundAst::Found(&InlinedItem::ImplItem(_, ref item)) => Some(item.id),
         _ => None
     };
     tcx.extern_const_fns.borrow_mut().insert(def_id,