diff options
| author | Matthias Krüger <matthias.krueger@famsik.de> | 2020-02-28 14:20:33 +0100 |
|---|---|---|
| committer | Matthias Krüger <matthias.krueger@famsik.de> | 2020-02-28 15:16:27 +0100 |
| commit | 9523c89f18ca62c992e580fc33fe1bf092696d8d (patch) | |
| tree | fe98aa841d6ff82631ffc933c6f632f16e96546f /src/librustc_codegen_ssa | |
| parent | e2223c94bf433fc38234d1303e88cbaf14755863 (diff) | |
| download | rust-9523c89f18ca62c992e580fc33fe1bf092696d8d.tar.gz rust-9523c89f18ca62c992e580fc33fe1bf092696d8d.zip | |
use is_empty() instead of len() == x to determine if structs are empty.
Diffstat (limited to 'src/librustc_codegen_ssa')
| -rw-r--r-- | src/librustc_codegen_ssa/back/link.rs | 4 | ||||
| -rw-r--r-- | src/librustc_codegen_ssa/back/write.rs | 12 |
2 files changed, 8 insertions, 8 deletions
diff --git a/src/librustc_codegen_ssa/back/link.rs b/src/librustc_codegen_ssa/back/link.rs index 78aacc56ea4..90601521b19 100644 --- a/src/librustc_codegen_ssa/back/link.rs +++ b/src/librustc_codegen_ssa/back/link.rs @@ -1524,12 +1524,12 @@ fn add_upstream_rust_crates<'a, B: ArchiveBuilder<'a>>( for &(cnum, _) in deps.iter().rev() { if let Some(missing) = info.missing_lang_items.get(&cnum) { end_with.extend(missing.iter().cloned()); - if end_with.len() > 0 && group_end.is_none() { + if !end_with.is_empty() && group_end.is_none() { group_end = Some(cnum); } } end_with.retain(|item| info.lang_item_to_crate.get(item) != Some(&cnum)); - if end_with.len() == 0 && group_end.is_some() { + if end_with.is_empty() && group_end.is_some() { group_start = Some(cnum); break; } diff --git a/src/librustc_codegen_ssa/back/write.rs b/src/librustc_codegen_ssa/back/write.rs index 92f795acc54..bc20ed6f6c5 100644 --- a/src/librustc_codegen_ssa/back/write.rs +++ b/src/librustc_codegen_ssa/back/write.rs @@ -1244,10 +1244,10 @@ fn start_executing_work<B: ExtraBackendMethods>( while !codegen_done || running > 0 || (!codegen_aborted - && (work_items.len() > 0 - || needs_fat_lto.len() > 0 - || needs_thin_lto.len() > 0 - || lto_import_only_modules.len() > 0 + && (!work_items.is_empty() + || !needs_fat_lto.is_empty() + || !needs_thin_lto.is_empty() + || !lto_import_only_modules.is_empty() || main_thread_worker_state != MainThreadWorkerState::Idle)) { // While there are still CGUs to be codegened, the coordinator has @@ -1289,7 +1289,7 @@ fn start_executing_work<B: ExtraBackendMethods>( // Perform the serial work here of figuring out what we're // going to LTO and then push a bunch of work items onto our // queue to do LTO - if work_items.len() == 0 + if work_items.is_empty() && running == 0 && main_thread_worker_state == MainThreadWorkerState::Idle { @@ -1354,7 +1354,7 @@ fn start_executing_work<B: ExtraBackendMethods>( // Spin up what work we can, only doing this while we've got available // parallelism slots and work left to spawn. - while !codegen_aborted && work_items.len() > 0 && running < tokens.len() { + while !codegen_aborted && !work_items.is_empty() && running < tokens.len() { let (item, _) = work_items.pop().unwrap(); maybe_start_llvm_timer(prof, cgcx.config(item.module_kind()), &mut llvm_start_time); |
