about summary refs log tree commit diff
diff options
context:
space:
mode:
authorNick Cameron <ncameron@mozilla.com>2016-04-07 18:16:12 +1200
committerNick Cameron <ncameron@mozilla.com>2016-04-20 10:13:35 +1200
commitc99b73a767a38d5b97214b7dbf6ee3c08ac782fd (patch)
tree8c0a506a47de619060c3a5e3e2028d5cd67c3317
parentc2aaad4e2288647c5235754a5e1439a5124978fe (diff)
downloadrust-c99b73a767a38d5b97214b7dbf6ee3c08ac782fd.tar.gz
rust-c99b73a767a38d5b97214b7dbf6ee3c08ac782fd.zip
Trivial refactoring
-rw-r--r--src/librustc_driver/driver.rs14
-rw-r--r--src/librustc_metadata/creader.rs17
2 files changed, 15 insertions, 16 deletions
diff --git a/src/librustc_driver/driver.rs b/src/librustc_driver/driver.rs
index de1a740e0bb..9c444fca9a1 100644
--- a/src/librustc_driver/driver.rs
+++ b/src/librustc_driver/driver.rs
@@ -766,12 +766,6 @@ pub fn phase_3_run_analysis_passes<'tcx, F, R>(sess: &'tcx Session,
          "external crate/lib resolution",
          || LocalCrateReader::new(sess, cstore, &hir_map, name).read_crates());
 
-    let lang_items = time(time_passes, "language item collection", || {
-        sess.track_errors(|| {
-            middle::lang_items::collect_language_items(&sess, &hir_map)
-        })
-    })?;
-
     let resolve::CrateMap {
         def_map,
         freevars,
@@ -779,9 +773,15 @@ pub fn phase_3_run_analysis_passes<'tcx, F, R>(sess: &'tcx Session,
         trait_map,
         glob_map,
     } = time(time_passes,
-             "resolution",
+             "name resolution",
              || resolve::resolve_crate(sess, &hir_map, make_glob_map));
 
+    let lang_items = time(time_passes, "language item collection", || {
+        sess.track_errors(|| {
+            middle::lang_items::collect_language_items(&sess, &hir_map)
+        })
+    })?;
+
     let mut analysis = ty::CrateAnalysis {
         export_map: export_map,
         access_levels: AccessLevels::default(),
diff --git a/src/librustc_metadata/creader.rs b/src/librustc_metadata/creader.rs
index df9072835b9..8ec238f4b24 100644
--- a/src/librustc_metadata/creader.rs
+++ b/src/librustc_metadata/creader.rs
@@ -813,9 +813,8 @@ impl<'a, 'b> LocalCrateReader<'a, 'b> {
 
     fn process_crate(&self, c: &hir::Crate) {
         for a in c.attrs.iter().filter(|m| m.name() == "link_args") {
-            match a.value_str() {
-                Some(ref linkarg) => self.cstore.add_used_link_args(&linkarg),
-                None => { /* fallthrough */ }
+            if let Some(ref linkarg) = a.value_str() {
+                self.cstore.add_used_link_args(&linkarg);
             }
         }
     }
@@ -830,12 +829,12 @@ impl<'a, 'b> LocalCrateReader<'a, 'b> {
                 match self.creader.extract_crate_info_hir(i) {
                     Some(info) => {
                         let (cnum, _, _) = self.creader.resolve_crate(&None,
-                                                                          &info.ident,
-                                                                          &info.name,
-                                                                          None,
-                                                                          i.span,
-                                                                          PathKind::Crate,
-                                                                          true);
+                                                                      &info.ident,
+                                                                      &info.name,
+                                                                      None,
+                                                                      i.span,
+                                                                      PathKind::Crate,
+                                                                      true);
                         let def_id = self.ast_map.local_def_id(i.id);
 
                         let len = self.ast_map.def_path(def_id).data.len();