Unrolled build for rust-lang#127652

rust-timer · web-flow · commit 20ccc80acafe · 2024-07-18T02:08:50.000-04:00
Rollup merge of rust-lang#127652 - GuillaumeGomez:cg-gcc-fmt, r=Urgau Unignore cg_gcc fmt `rustc_codegen_gcc` uses `rustfmt` now so it can be unignored. r? ``@Urgau``
diff --git a/compiler/rustc_codegen_gcc/.rustfmt.toml b/compiler/rustc_codegen_gcc/.rustfmt.toml
@@ -1 +1,3 @@
+version = "Two"
 use_small_heuristics = "Max"
+merge_derives = false
diff --git a/compiler/rustc_codegen_gcc/build_system/src/clone_gcc.rs b/compiler/rustc_codegen_gcc/build_system/src/clone_gcc.rs
@@ -34,7 +34,7 @@ impl Args {
                 "--out-path" => match args.next() {
                     Some(path) if !path.is_empty() => out_path = Some(path),
                     _ => {
-                        return Err("Expected an argument after `--out-path`, found nothing".into())
+                        return Err("Expected an argument after `--out-path`, found nothing".into());
                     }
                 },
                 "--help" => {
diff --git a/compiler/rustc_codegen_gcc/build_system/src/config.rs b/compiler/rustc_codegen_gcc/build_system/src/config.rs
@@ -54,7 +54,7 @@ impl ConfigFile {
                     config.gcc_path = Some(value.as_str().to_string())
                 }
                 ("gcc-path", _) => {
-                    return failed_config_parsing(config_file, "Expected a string for `gcc-path`")
+                    return failed_config_parsing(config_file, "Expected a string for `gcc-path`");
                 }
                 ("download-gccjit", TomlValue::Boolean(value)) => {
                     config.download_gccjit = Some(*value)
@@ -63,7 +63,7 @@ impl ConfigFile {
                     return failed_config_parsing(
                         config_file,
                         "Expected a boolean for `download-gccjit`",
-                    )
+                    );
                 }
                 _ => return failed_config_parsing(config_file, &format!("Unknown key `{}`", key)),
             }
@@ -73,7 +73,7 @@ impl ConfigFile {
                 return failed_config_parsing(
                     config_file,
                     "At least one of `gcc-path` or `download-gccjit` value must be set",
-                )
+                );
             }
             (Some(_), Some(true)) => {
                 println!(
@@ -144,7 +144,7 @@ impl ConfigInfo {
                 _ => {
                     return Err(
                         "Expected a value after `--target-triple`, found nothing".to_string()
-                    )
+                    );
                 }
             },
             "--out-dir" => match args.next() {
@@ -158,7 +158,7 @@ impl ConfigInfo {
                     self.config_file = Some(arg.to_string());
                 }
                 _ => {
-                    return Err("Expected a value after `--config-file`, found nothing".to_string())
+                    return Err("Expected a value after `--config-file`, found nothing".to_string());
                 }
             },
             "--release-sysroot" => self.sysroot_release_channel = true,
@@ -169,7 +169,7 @@ impl ConfigInfo {
                     self.cg_gcc_path = Some(arg.into());
                 }
                 _ => {
-                    return Err("Expected a value after `--cg_gcc-path`, found nothing".to_string())
+                    return Err("Expected a value after `--cg_gcc-path`, found nothing".to_string());
                 }
             },
             "--use-backend" => match args.next() {
@@ -277,7 +277,7 @@ impl ConfigInfo {
         self.gcc_path = match gcc_path {
             Some(path) => path,
             None => {
-                return Err(format!("missing `gcc-path` value from `{}`", config_file.display(),))
+                return Err(format!("missing `gcc-path` value from `{}`", config_file.display(),));
             }
         };
         Ok(())
diff --git a/compiler/rustc_codegen_gcc/build_system/src/test.rs b/compiler/rustc_codegen_gcc/build_system/src/test.rs
@@ -109,7 +109,7 @@ impl TestArg {
                         test_arg.flags.extend_from_slice(&["--features".into(), feature]);
                     }
                     _ => {
-                        return Err("Expected an argument after `--features`, found nothing".into())
+                        return Err("Expected an argument after `--features`, found nothing".into());
                     }
                 },
                 "--use-system-gcc" => {
@@ -458,11 +458,7 @@ fn setup_rustc(env: &mut Env, args: &TestArg) -> Result<PathBuf, String> {
     .map_err(|error| format!("Failed to retrieve cargo path: {:?}", error))
     .and_then(|cargo| {
         let cargo = cargo.trim().to_owned();
-        if cargo.is_empty() {
-            Err(format!("`cargo` path is empty"))
-        } else {
-            Ok(cargo)
-        }
+        if cargo.is_empty() { Err(format!("`cargo` path is empty")) } else { Ok(cargo) }
     })?;
     let rustc = String::from_utf8(
         run_command_with_env(&[&"rustup", &toolchain, &"which", &"rustc"], rust_dir, Some(env))?
@@ -471,11 +467,7 @@ fn setup_rustc(env: &mut Env, args: &TestArg) -> Result<PathBuf, String> {
     .map_err(|error| format!("Failed to retrieve rustc path: {:?}", error))
     .and_then(|rustc| {
         let rustc = rustc.trim().to_owned();
-        if rustc.is_empty() {
-            Err(format!("`rustc` path is empty"))
-        } else {
-            Ok(rustc)
-        }
+        if rustc.is_empty() { Err(format!("`rustc` path is empty")) } else { Ok(rustc) }
     })?;
     let llvm_filecheck = match run_command_with_env(
         &[
diff --git a/compiler/rustc_codegen_gcc/build_system/src/utils.rs b/compiler/rustc_codegen_gcc/build_system/src/utils.rs
@@ -175,11 +175,7 @@ pub fn cargo_install(to_install: &str) -> Result<(), String> {
 pub fn get_os_name() -> Result<String, String> {
     let output = run_command(&[&"uname"], None)?;
     let name = std::str::from_utf8(&output.stdout).unwrap_or("").trim().to_string();
-    if !name.is_empty() {
-        Ok(name)
-    } else {
-        Err("Failed to retrieve the OS name".to_string())
-    }
+    if !name.is_empty() { Ok(name) } else { Err("Failed to retrieve the OS name".to_string()) }
 }
 
 #[derive(Default, PartialEq)]
diff --git a/compiler/rustc_codegen_gcc/src/abi.rs b/compiler/rustc_codegen_gcc/src/abi.rs
@@ -26,11 +26,7 @@ impl<'a, 'gcc, 'tcx> AbiBuilderMethods<'tcx> for Builder<'a, 'gcc, 'tcx> {
         } else {
             false
         };
-        if on_stack {
-            param.to_lvalue().get_address(None)
-        } else {
-            param.to_rvalue()
-        }
+        if on_stack { param.to_lvalue().get_address(None) } else { param.to_rvalue() }
     }
 }
 
diff --git a/compiler/rustc_codegen_gcc/src/asm.rs b/compiler/rustc_codegen_gcc/src/asm.rs
@@ -858,11 +858,7 @@ fn modifier_to_gcc(
         InlineAsmRegClass::AArch64(AArch64InlineAsmRegClass::reg) => modifier,
         InlineAsmRegClass::AArch64(AArch64InlineAsmRegClass::vreg)
         | InlineAsmRegClass::AArch64(AArch64InlineAsmRegClass::vreg_low16) => {
-            if modifier == Some('v') {
-                None
-            } else {
-                modifier
-            }
+            if modifier == Some('v') { None } else { modifier }
         }
         InlineAsmRegClass::AArch64(AArch64InlineAsmRegClass::preg) => {
             unreachable!("clobber-only")
diff --git a/compiler/rustc_codegen_gcc/src/builder.rs b/compiler/rustc_codegen_gcc/src/builder.rs
@@ -1043,11 +1043,7 @@ impl<'a, 'gcc, 'tcx> BuilderMethods<'a, 'tcx> for Builder<'a, 'gcc, 'tcx> {
                 let llty = place.layout.scalar_pair_element_gcc_type(self, i);
                 let load = self.load(llty, llptr, align);
                 scalar_load_metadata(self, load, scalar);
-                if scalar.is_bool() {
-                    self.trunc(load, self.type_i1())
-                } else {
-                    load
-                }
+                if scalar.is_bool() { self.trunc(load, self.type_i1()) } else { load }
             };
 
             OperandValue::Pair(
@@ -1795,18 +1791,10 @@ impl<'a, 'gcc, 'tcx> Builder<'a, 'gcc, 'tcx> {
         // This already happens today with u128::MAX = 2^128 - 1 > f32::MAX.
         let int_max = |signed: bool, int_width: u64| -> u128 {
             let shift_amount = 128 - int_width;
-            if signed {
-                i128::MAX as u128 >> shift_amount
-            } else {
-                u128::MAX >> shift_amount
-            }
+            if signed { i128::MAX as u128 >> shift_amount } else { u128::MAX >> shift_amount }
         };
         let int_min = |signed: bool, int_width: u64| -> i128 {
-            if signed {
-                i128::MIN >> (128 - int_width)
-            } else {
-                0
-            }
+            if signed { i128::MIN >> (128 - int_width) } else { 0 }
         };
 
         let compute_clamp_bounds_single = |signed: bool, int_width: u64| -> (u128, u128) {
diff --git a/compiler/rustc_codegen_gcc/src/common.rs b/compiler/rustc_codegen_gcc/src/common.rs
@@ -58,11 +58,7 @@ pub fn type_is_pointer(typ: Type<'_>) -> bool {
 
 impl<'gcc, 'tcx> ConstMethods<'tcx> for CodegenCx<'gcc, 'tcx> {
     fn const_null(&self, typ: Type<'gcc>) -> RValue<'gcc> {
-        if type_is_pointer(typ) {
-            self.context.new_null(typ)
-        } else {
-            self.const_int(typ, 0)
-        }
+        if type_is_pointer(typ) { self.context.new_null(typ) } else { self.const_int(typ, 0) }
     }
 
     fn const_undef(&self, typ: Type<'gcc>) -> RValue<'gcc> {
diff --git a/rustfmt.toml b/rustfmt.toml
@@ -31,7 +31,6 @@ ignore = [
     "library/backtrace",
     "library/portable-simd",
     "library/stdarch",
-    "compiler/rustc_codegen_gcc",
     "src/doc/book",
     "src/doc/edition-guide",
     "src/doc/embedded-book",
@@ -50,4 +49,8 @@ ignore = [
     # These are ignored by a standard cargo fmt run.
     "compiler/rustc_codegen_cranelift/scripts",
     "compiler/rustc_codegen_cranelift/example/gen_block_iterate.rs", # uses edition 2024
+    "compiler/rustc_codegen_gcc/tests",
+    # Code automatically generated and included.
+    "compiler/rustc_codegen_gcc/src/intrinsic/archs.rs",
+    "compiler/rustc_codegen_gcc/example",
 ]

Original file line number	Diff line number	Diff line change
`@@ -1 +1,3 @@`
	`1`	`+version = "Two"`
`1`	`2`	`use_small_heuristics = "Max"`
	`3`	`+merge_derives = false`
Original file line number	Diff line number	Diff line change
`@@ -34,7 +34,7 @@ impl Args {`
`34`	`34`	`"--out-path" => match args.next() {`
`35`	`35`	`Some(path) if !path.is_empty() => out_path = Some(path),`
`36`	`36`	`_ => {`
`37`		- return Err("Expected an argument after `--out-path`, found nothing".into())
	`37`	+ return Err("Expected an argument after `--out-path`, found nothing".into());
`38`	`38`	`}`
`39`	`39`	`},`
`40`	`40`	`"--help" => {`
Original file line number	Diff line number	Diff line change
`@@ -54,7 +54,7 @@ impl ConfigFile {`
`54`	`54`	`config.gcc_path = Some(value.as_str().to_string())`
`55`	`55`	`}`
`56`	`56`	`("gcc-path", _) => {`
`57`		- return failed_config_parsing(config_file, "Expected a string for `gcc-path`")
	`57`	+ return failed_config_parsing(config_file, "Expected a string for `gcc-path`");
`58`	`58`	`}`
`59`	`59`	`("download-gccjit", TomlValue::Boolean(value)) => {`
`60`	`60`	`config.download_gccjit = Some(*value)`
`@@ -63,7 +63,7 @@ impl ConfigFile {`
`63`	`63`	`return failed_config_parsing(`
`64`	`64`	`config_file,`
`65`	`65`	"Expected a boolean for `download-gccjit`",
`66`		`- )`
	`66`	`+ );`
`67`	`67`	`}`
`68`	`68`	_ => return failed_config_parsing(config_file, &format!("Unknown key `{}`", key)),
`69`	`69`	`}`
`@@ -73,7 +73,7 @@ impl ConfigFile {`
`73`	`73`	`return failed_config_parsing(`
`74`	`74`	`config_file,`
`75`	`75`	"At least one of `gcc-path` or `download-gccjit` value must be set",
`76`		`- )`
	`76`	`+ );`
`77`	`77`	`}`
`78`	`78`	`(Some(_), Some(true)) => {`
`79`	`79`	`println!(`
`@@ -144,7 +144,7 @@ impl ConfigInfo {`
`144`	`144`	`_ => {`
`145`	`145`	`return Err(`
`146`	`146`	"Expected a value after `--target-triple`, found nothing".to_string()
`147`		`- )`
	`147`	`+ );`
`148`	`148`	`}`
`149`	`149`	`},`
`150`	`150`	`"--out-dir" => match args.next() {`
`@@ -158,7 +158,7 @@ impl ConfigInfo {`
`158`	`158`	`self.config_file = Some(arg.to_string());`
`159`	`159`	`}`
`160`	`160`	`_ => {`
`161`		- return Err("Expected a value after `--config-file`, found nothing".to_string())
	`161`	+ return Err("Expected a value after `--config-file`, found nothing".to_string());
`162`	`162`	`}`
`163`	`163`	`},`
`164`	`164`	`"--release-sysroot" => self.sysroot_release_channel = true,`
`@@ -169,7 +169,7 @@ impl ConfigInfo {`
`169`	`169`	`self.cg_gcc_path = Some(arg.into());`
`170`	`170`	`}`
`171`	`171`	`_ => {`
`172`		- return Err("Expected a value after `--cg_gcc-path`, found nothing".to_string())
	`172`	+ return Err("Expected a value after `--cg_gcc-path`, found nothing".to_string());
`173`	`173`	`}`
`174`	`174`	`},`
`175`	`175`	`"--use-backend" => match args.next() {`
`@@ -277,7 +277,7 @@ impl ConfigInfo {`
`277`	`277`	`self.gcc_path = match gcc_path {`
`278`	`278`	`Some(path) => path,`
`279`	`279`	`None => {`
`280`		- return Err(format!("missing `gcc-path` value from `{}`", config_file.display(),))
	`280`	+ return Err(format!("missing `gcc-path` value from `{}`", config_file.display(),));
`281`	`281`	`}`
`282`	`282`	`};`
`283`	`283`	`Ok(())`
Original file line number	Diff line number	Diff line change
`@@ -26,11 +26,7 @@ impl<'a, 'gcc, 'tcx> AbiBuilderMethods<'tcx> for Builder<'a, 'gcc, 'tcx> {`
`26`	`26`	`} else {`
`27`	`27`	`false`
`28`	`28`	`};`
`29`		`- if on_stack {`
`30`		`- param.to_lvalue().get_address(None)`
`31`		`- } else {`
`32`		`- param.to_rvalue()`
`33`		`- }`
	`29`	`+ if on_stack { param.to_lvalue().get_address(None) } else { param.to_rvalue() }`
`34`	`30`	`}`
`35`	`31`	`}`
`36`	`32`