From 4e2189fe457e246654f6da6152da1609b392aeea Mon Sep 17 00:00:00 2001 From: Seo Sanghyeon Date: Tue, 27 Oct 2015 01:32:04 +0900 Subject: [PATCH] Remove NodeArg --- src/librustc/front/map/collector.rs | 17 ----------------- src/librustc/front/map/mod.rs | 11 +---------- src/librustc/middle/mem_categorization.rs | 2 +- src/librustc_trans/trans/debuginfo/metadata.rs | 2 +- src/librustc_trans/trans/monomorphize.rs | 1 - 5 files changed, 3 insertions(+), 30 deletions(-) diff --git a/src/librustc/front/map/collector.rs b/src/librustc/front/map/collector.rs index fc7531d3f61..40382d7c771 100644 --- a/src/librustc/front/map/collector.rs +++ b/src/librustc/front/map/collector.rs @@ -104,12 +104,6 @@ impl<'ast> NodeCollector<'ast> { let entry = MapEntry::from_node(self.parent_node, node); self.insert_entry(id, entry); } - - fn visit_fn_decl(&mut self, decl: &'ast FnDecl) { - for a in &decl.inputs { - self.insert(a.id, NodeArg(&*a.pat)); - } - } } impl<'ast> Visitor<'ast> for NodeCollector<'ast> { @@ -295,20 +289,9 @@ impl<'ast> Visitor<'ast> for NodeCollector<'ast> { fn visit_fn(&mut self, fk: visit::FnKind<'ast>, fd: &'ast FnDecl, b: &'ast Block, s: Span, id: NodeId) { assert_eq!(self.parent_node, id); - self.visit_fn_decl(fd); visit::walk_fn(self, fk, fd, b, s); } - fn visit_ty(&mut self, ty: &'ast Ty) { - match ty.node { - TyBareFn(ref fd) => { - self.visit_fn_decl(&*fd.decl); - } - _ => {} - } - visit::walk_ty(self, ty); - } - fn visit_block(&mut self, block: &'ast Block) { self.insert(block.id, NodeBlock(block)); let parent_node = self.parent_node; diff --git a/src/librustc/front/map/mod.rs b/src/librustc/front/map/mod.rs index 68ed0cda5f1..94800db184e 100644 --- a/src/librustc/front/map/mod.rs +++ b/src/librustc/front/map/mod.rs @@ -118,7 +118,6 @@ pub enum Node<'ast> { NodeVariant(&'ast Variant), NodeExpr(&'ast Expr), NodeStmt(&'ast Stmt), - NodeArg(&'ast Pat), NodeLocal(&'ast Pat), NodePat(&'ast Pat), NodeBlock(&'ast Block), @@ -145,7 +144,6 @@ pub enum MapEntry<'ast> { EntryVariant(NodeId, &'ast Variant), EntryExpr(NodeId, &'ast Expr), EntryStmt(NodeId, &'ast Stmt), - EntryArg(NodeId, &'ast Pat), EntryLocal(NodeId, &'ast Pat), EntryPat(NodeId, &'ast Pat), EntryBlock(NodeId, &'ast Block), @@ -180,7 +178,6 @@ impl<'ast> MapEntry<'ast> { NodeVariant(n) => EntryVariant(p, n), NodeExpr(n) => EntryExpr(p, n), NodeStmt(n) => EntryStmt(p, n), - NodeArg(n) => EntryArg(p, n), NodeLocal(n) => EntryLocal(p, n), NodePat(n) => EntryPat(p, n), NodeBlock(n) => EntryBlock(p, n), @@ -199,7 +196,6 @@ impl<'ast> MapEntry<'ast> { EntryVariant(id, _) => id, EntryExpr(id, _) => id, EntryStmt(id, _) => id, - EntryArg(id, _) => id, EntryLocal(id, _) => id, EntryPat(id, _) => id, EntryBlock(id, _) => id, @@ -219,7 +215,6 @@ impl<'ast> MapEntry<'ast> { EntryVariant(_, n) => NodeVariant(n), EntryExpr(_, n) => NodeExpr(n), EntryStmt(_, n) => NodeStmt(n), - EntryArg(_, n) => NodeArg(n), EntryLocal(_, n) => NodeLocal(n), EntryPat(_, n) => NodePat(n), EntryBlock(_, n) => NodeBlock(n), @@ -649,7 +644,7 @@ impl<'ast> Map<'ast> { Some(NodeVariant(variant)) => variant.span, Some(NodeExpr(expr)) => expr.span, Some(NodeStmt(stmt)) => stmt.span, - Some(NodeArg(pat)) | Some(NodeLocal(pat)) => pat.span, + Some(NodeLocal(pat)) => pat.span, Some(NodePat(pat)) => pat.span, Some(NodeBlock(block)) => block.span, Some(NodeStructCtor(_)) => self.expect_item(self.get_parent(id)).span, @@ -907,7 +902,6 @@ impl<'a> NodePrinter for pprust::State<'a> { // ast_map to reconstruct their full structure for pretty // printing. NodeLocal(_) => panic!("cannot print isolated Local"), - NodeArg(_) => panic!("cannot print isolated Arg"), NodeStructCtor(_) => panic!("cannot print isolated StructCtor"), } } @@ -986,9 +980,6 @@ fn node_id_to_string(map: &Map, id: NodeId, include_id: bool) -> String { Some(NodeStmt(ref stmt)) => { format!("stmt {}{}", pprust::stmt_to_string(&**stmt), id_str) } - Some(NodeArg(ref pat)) => { - format!("arg {}{}", pprust::pat_to_string(&**pat), id_str) - } Some(NodeLocal(ref pat)) => { format!("local {}{}", pprust::pat_to_string(&**pat), id_str) } diff --git a/src/librustc/middle/mem_categorization.rs b/src/librustc/middle/mem_categorization.rs index 186a8b1390e..1fcd6e92305 100644 --- a/src/librustc/middle/mem_categorization.rs +++ b/src/librustc/middle/mem_categorization.rs @@ -305,7 +305,7 @@ impl MutabilityCategory { fn from_local(tcx: &ty::ctxt, id: ast::NodeId) -> MutabilityCategory { let ret = match tcx.map.get(id) { - ast_map::NodeLocal(p) | ast_map::NodeArg(p) => match p.node { + ast_map::NodeLocal(p) => match p.node { hir::PatIdent(bind_mode, _, _) => { if bind_mode == hir::BindByValue(hir::MutMutable) { McDeclared diff --git a/src/librustc_trans/trans/debuginfo/metadata.rs b/src/librustc_trans/trans/debuginfo/metadata.rs index 62befb3b992..2e7b1f31ba9 100644 --- a/src/librustc_trans/trans/debuginfo/metadata.rs +++ b/src/librustc_trans/trans/debuginfo/metadata.rs @@ -1976,7 +1976,7 @@ pub fn create_captured_var_metadata<'blk, 'tcx>(bcx: Block<'blk, 'tcx>, None => { cx.sess().span_bug(span, "debuginfo::create_captured_var_metadata: node not found"); } - Some(hir_map::NodeLocal(pat)) | Some(hir_map::NodeArg(pat)) => { + Some(hir_map::NodeLocal(pat)) => { match pat.node { hir::PatIdent(_, ref path1, _) => { path1.node.name diff --git a/src/librustc_trans/trans/monomorphize.rs b/src/librustc_trans/trans/monomorphize.rs index f84f0feb960..1f9116f7491 100644 --- a/src/librustc_trans/trans/monomorphize.rs +++ b/src/librustc_trans/trans/monomorphize.rs @@ -262,7 +262,6 @@ pub fn monomorphic_fn<'a, 'tcx>(ccx: &CrateContext<'a, 'tcx>, hir_map::NodeTyParam(..) | hir_map::NodeExpr(..) | hir_map::NodeStmt(..) | - hir_map::NodeArg(..) | hir_map::NodeBlock(..) | hir_map::NodePat(..) | hir_map::NodeLocal(..) => {