Fix warnings

Author: antoyo

Cargo.lock

@@ -14,17 +14,16 @@ path = "tests/lib.rs"
 harness = false
 
 [dependencies]
-#gccjit = { git = "https://github.com/antoyo/gccjit.rs", branch = "feature/rustc" }
-gccjit = { path = "/home/bouanto/Ordinateur/Programmation/Rust/Projets/gccjit.rs" }
+gccjit = { git = "https://github.com/antoyo/gccjit.rs" }
 
 target-lexicon = "0.10.0"
 
 ar = "0.8.0"
 
 [dependencies.object]
-version = "0.18.0"
+version = "0.25.0"
 default-features = false
-features = ["read", "std", "write"] # We don't need WASM support
+features = ["read", "std", "write"] # We don't need WASM support.
 
 [dev-dependencies]
 lang_tester = "0.3.9"

Cargo.toml

@@ -407,7 +407,7 @@ pub trait FnMut<Args>: FnOnce<Args> {
 
 #[lang = "panic"]
 #[track_caller]
-pub fn panic(msg: &str) -> ! {
+pub fn panic(_msg: &str) -> ! {
     unsafe {
         libc::puts("Panicking\n\0" as *const str as *const u8);
         intrinsics::abort();

example/mini_core.rs

@@ -415,10 +415,10 @@ pub enum E2<X> {
 
 fn check_niche_behavior () {
     if let E1::V2 { .. } = (E1::V1 { f: true }) {
-        unsafe { intrinsics::abort(); }
+        intrinsics::abort();
     }
 
     if let E2::V1 { .. } = E2::V3::<Infallible> {
-        unsafe { intrinsics::abort(); }
+        intrinsics::abort();
     }
 }

example/mini_core_hello_world.rs

@@ -132,7 +132,7 @@ unsafe fn test_simd() {
     assert_eq!(mask1, 1);*/
 }*/
 
-#[target_feature(enable = "sse2")]
+/*#[target_feature(enable = "sse2")]
 unsafe fn test_mm_slli_si128() {
     #[rustfmt::skip]
     let a = _mm_setr_epi8(
@@ -258,7 +258,7 @@ unsafe fn test_mm_extract_epi8() {
     let r2 = _mm_extract_epi8(a, 19);
     assert_eq!(r1, 0xFF);
     assert_eq!(r2, 3);
-}
+}*/
 
 #[derive(PartialEq)]
 enum LoopState {

example/std_example.rs

@@ -2,8 +2,6 @@
 
 // run-pass
 
-#![feature(track_caller)]
-
 use std::panic::Location;
 
 #[track_caller]
@@ -22,21 +20,21 @@ fn nested_tracked() -> &'static Location<'static> {
 fn main() {
     let location = Location::caller();
     assert_eq!(location.file(), file!());
-    assert_eq!(location.line(), 23);
+    assert_eq!(location.line(), 21);
     assert_eq!(location.column(), 20);
 
     let tracked = tracked();
     assert_eq!(tracked.file(), file!());
-    assert_eq!(tracked.line(), 28);
+    assert_eq!(tracked.line(), 26);
     assert_eq!(tracked.column(), 19);
 
     let nested = nested_intrinsic();
     assert_eq!(nested.file(), file!());
-    assert_eq!(nested.line(), 15);
+    assert_eq!(nested.line(), 13);
     assert_eq!(nested.column(), 5);
 
     let contained = nested_tracked();
     assert_eq!(contained.file(), file!());
-    assert_eq!(contained.line(), 19);
+    assert_eq!(contained.line(), 17);
     assert_eq!(contained.column(), 5);
 }

example/track-caller-attribute.rs

@@ -109,7 +109,7 @@ pub(crate) unsafe fn codegen(tcx: TyCtxt<'_>, mods: &mut GccContext, kind: Alloc
         .enumerate()
         .map(|(i, _)| func.get_param(i as i32).to_rvalue())
         .collect::<Vec<_>>();
-    let ret = context.new_call(None, callee, &args);
+    let _ret = context.new_call(None, callee, &args);
     //llvm::LLVMSetTailCall(ret, True);
     block.end_with_void_return(None);
 }

src/allocator.rs

@@ -31,7 +31,6 @@ pub struct ArArchiveBuilder<'a> {
     // Don't use `HashMap` here, as the order is important. `rust.metadata.bin` must always be at
     // the end of an archive for linkers to not get confused.
     entries: Vec<(String, ArchiveEntry)>,
-    update_symbols: bool,
 }
 
 impl<'a> ArchiveBuilder<'a> for ArArchiveBuilder<'a> {
@@ -72,7 +71,6 @@ impl<'a> ArchiveBuilder<'a> for ArArchiveBuilder<'a> {
             config,
             src_archives,
             entries,
-            update_symbols: false,
         }
     }
 
@@ -140,7 +138,6 @@ impl<'a> ArchiveBuilder<'a> for ArArchiveBuilder<'a> {
     }
 
     fn update_symbols(&mut self) {
-        self.update_symbols = true;
     }
 
     fn build(mut self) {
@@ -238,7 +235,7 @@ impl<'a> ArchiveBuilder<'a> for ArArchiveBuilder<'a> {
         }
     }
 
-    fn inject_dll_import_lib(&mut self, lib_name: &str, dll_imports: &[DllImport], tmpdir: &MaybeTempDir) {
+    fn inject_dll_import_lib(&mut self, _lib_name: &str, _dll_imports: &[DllImport], _tmpdir: &MaybeTempDir) {
         unimplemented!();
     }
 }

src/archive.rs

@@ -117,7 +117,7 @@ impl<'a, 'gcc, 'tcx> AsmBuilderMethods<'tcx> for Builder<'a, 'gcc, 'tcx> {
         true*/
     }
 
-    fn codegen_inline_asm(&mut self, template: &[InlineAsmTemplatePiece], operands: &[InlineAsmOperandRef<'tcx, Self>], options: InlineAsmOptions, span: &[Span]) {
+    fn codegen_inline_asm(&mut self, template: &[InlineAsmTemplatePiece], operands: &[InlineAsmOperandRef<'tcx, Self>], options: InlineAsmOptions, _span: &[Span]) {
         let asm_arch = self.tcx.sess.asm_arch.unwrap();
 
         let intel_dialect =
@@ -135,7 +135,7 @@ impl<'a, 'gcc, 'tcx> AsmBuilderMethods<'tcx> for Builder<'a, 'gcc, 'tcx> {
         let mut current_number = 0;
         for (idx, op) in operands.iter().enumerate() {
             match *op {
-                InlineAsmOperandRef::Out { reg, late, place } => {
+                InlineAsmOperandRef::Out { place, .. } => {
                     let ty =
                         match place {
                             Some(place) => place.layout.gcc_type(self.cx, false),
@@ -155,7 +155,7 @@ impl<'a, 'gcc, 'tcx> AsmBuilderMethods<'tcx> for Builder<'a, 'gcc, 'tcx> {
                     current_number += 1;
                     output_vars.insert(idx, var);
                 }
-                InlineAsmOperandRef::InOut { reg, late, out_place, .. } => {
+                InlineAsmOperandRef::InOut { out_place, .. } => {
                     let ty =
                         match out_place {
                             Some(place) => place.layout.gcc_type(self.cx, false),
@@ -218,7 +218,7 @@ impl<'a, 'gcc, 'tcx> AsmBuilderMethods<'tcx> for Builder<'a, 'gcc, 'tcx> {
                                 template_str.push_str(&format!("%{}", operand_numbers[&operand_idx]));
                             }
                         },
-                        InlineAsmOperandRef::Out { reg, place: None, .. } => {
+                        InlineAsmOperandRef::Out { place: None, .. } => {
                             unimplemented!("Out None");
                         },
                         InlineAsmOperandRef::In { reg, .. }
@@ -321,35 +321,6 @@ impl<'a, 'gcc, 'tcx> AsmBuilderMethods<'tcx> for Builder<'a, 'gcc, 'tcx> {
             }
         }
 
-        // Collect input operands
-        //let mut constraints = vec![];
-        //let mut op_idx = FxHashMap::default();
-        //let mut inputs = vec![];
-        for (idx, op) in operands.iter().enumerate() {
-            match *op {
-                InlineAsmOperandRef::In { reg, value } => {
-                    //inputs.push(value.immediate());
-                    //op_idx.insert(idx, constraints.len());
-                    //constraints.push(reg_to_gcc(reg));
-                }
-                InlineAsmOperandRef::InOut { reg, late: _, in_value, out_place: _ } => {
-                    //inputs.push(in_value.immediate());
-                    //constraints.push(format!("{}", op_idx[&idx]));
-                }
-                InlineAsmOperandRef::SymFn { instance } => {
-                    //inputs.push(self.cx.get_fn(instance));
-                    //op_idx.insert(idx, constraints.len());
-                    //constraints.push("s".to_string());
-                }
-                InlineAsmOperandRef::SymStatic { def_id } => {
-                    //inputs.push(self.cx.get_static(def_id));
-                    //op_idx.insert(idx, constraints.len());
-                    //constraints.push("s".to_string());
-                }
-                _ => {}
-            }
-        }
-
         /*if !options.contains(InlineAsmOptions::PRESERVES_FLAGS) {
             match asm_arch {
                 InlineAsmArch::AArch64 | InlineAsmArch::Arm => {
@@ -409,8 +380,8 @@ impl<'a, 'gcc, 'tcx> AsmBuilderMethods<'tcx> for Builder<'a, 'gcc, 'tcx> {
 
         // Write results to outputs
         for (idx, op) in operands.iter().enumerate() {
-            if let InlineAsmOperandRef::Out { reg, place: Some(place), .. }
-            | InlineAsmOperandRef::InOut { reg, out_place: Some(place), .. } = *op
+            if let InlineAsmOperandRef::Out { place: Some(place), .. }
+            | InlineAsmOperandRef::InOut { out_place: Some(place), .. } = *op
             {
                 OperandValue::Immediate(output_vars[&idx].to_rvalue()).store(self, place);
             }
@@ -628,9 +599,9 @@ fn modifier_to_gcc(arch: InlineAsmArch, reg: InlineAsmRegClass, modifier: Option
             _ => unreachable!(),
         },
         InlineAsmRegClass::X86(X86InlineAsmRegClass::reg_byte) => unimplemented!(),
-        InlineAsmRegClass::X86(reg @ X86InlineAsmRegClass::xmm_reg)
-        | InlineAsmRegClass::X86(reg @ X86InlineAsmRegClass::ymm_reg)
-        | InlineAsmRegClass::X86(reg @ X86InlineAsmRegClass::zmm_reg) => unimplemented!() /*match (reg, modifier) {
+        InlineAsmRegClass::X86(X86InlineAsmRegClass::xmm_reg)
+        | InlineAsmRegClass::X86(X86InlineAsmRegClass::ymm_reg)
+        | InlineAsmRegClass::X86(X86InlineAsmRegClass::zmm_reg) => unimplemented!() /*match (reg, modifier) {
             (X86InlineAsmRegClass::xmm_reg, None) => Some('x'),
             (X86InlineAsmRegClass::ymm_reg, None) => Some('t'),
             (X86InlineAsmRegClass::zmm_reg, None) => Some('g'),