Refactoring a bit

Cargo.lock

@@ -1081,14 +1081,6 @@ dependencies = [
  "drop_bomb",
 ]
 
-[[package]]
-name = "ra_proc_macro"
-version = "0.1.0"
-dependencies = [
- "ra_mbe",
- "ra_tt",
-]
-
 [[package]]
 name = "ra_prof"
 version = "0.1.0"

crates/ra_hir_def/src/lib.rs

@@ -476,7 +476,10 @@ impl AsMacroCall for AstIdWithPath<ast::ModuleItem> {
     ) -> Option<MacroCallId> {
         let def = resolver(self.path.clone())?;
         Some(
-            def.as_lazy_macro(db.upcast(), MacroCallKind::Attr(self.ast_id, self.path.to_string()))
+            def.as_lazy_macro(
+                db.upcast(),
+                MacroCallKind::Attr(self.ast_id, self.path.segments.last()?.to_string()),
+            )
             .into(),
         )
     }

crates/ra_hir_expand/src/builtin_derive.rs

@@ -229,7 +229,7 @@ fn partial_ord_expand(
 mod tests {
     use super::*;
     use crate::{test_db::TestDB, AstId, MacroCallId, MacroCallKind, MacroCallLoc};
-    use name::Name;
+    use name::{known, Name};
     use ra_db::{fixture::WithFixture, SourceDatabase};
 
     fn expand_builtin_derive(s: &str, name: Name) -> String {
@@ -261,7 +261,7 @@ mod tests {
         #[derive(Copy)]
         struct Foo;
 "#,
-            name::known::Copy,
+            known::Copy,
         );
 
         assert_eq!(expanded, "impl< >std::marker::CopyforFoo< >{}");
@@ -274,7 +274,7 @@ mod tests {
         #[derive(Copy)]
         struct Foo<A, B>;
 "#,
-            name::known::Copy,
+            known::Copy,
         );
 
         assert_eq!(
@@ -290,7 +290,7 @@ mod tests {
         #[derive(Copy)]
         struct Foo<A, B, 'a, 'b>;
 "#,
-            name::known::Copy,
+            known::Copy,
         );
 
         // We currently just ignore lifetimes
@@ -308,7 +308,7 @@ mod tests {
         #[derive(Clone)]
         struct Foo<A, B>;
 "#,
-            name::known::Clone,
+            known::Clone,
         );
 
         assert_eq!(