Refactor pass infra

scripts/execute.py

@@ -327,18 +327,18 @@ def dfs(curr_dir: str):
         log_path = os.path.join(output_dir, f"{basename}.log")
         log_file = open(log_path, "w")
 
-        command = (
-            f"clang --target=riscv64 -march=rv64imafdc_zba_zbb -w -xc -O3 -S {testcase}.sy"
-            f" -o {clang_asm_path}"
-        )
-
-        exec_result = execute(command, exec_timeout)
-        log(log_file, command, exec_result)
-
-        if exec_result["returncode"] is None or exec_result["stderr"] != "":
-            result_md_table += f"| `{testcase}` | 😢 CE |\n"
-            print(f"\033[33m[  ERROR  ] (clang CE)\033[0m {testcase}, see: ", log_path)
-            continue
+        # command = (
+        #     f"clang --target=riscv64 -march=rv64imafdc_zba_zbb -w -xc -Wno-implicit-function-declaration -O3 -S {testcase}.sy"
+        #     f" -o {clang_asm_path}"
+        # )
+
+        # exec_result = execute(command, exec_timeout)
+        # log(log_file, command, exec_result)
+
+        # if exec_result["returncode"] is None or exec_result["stderr"] != "":
+        #     result_md_table += f"| `{testcase}` | 😢 CE |\n"
+        #     print(f"\033[33m[  ERROR  ] (clang CE)\033[0m {testcase}, see: ", log_path)
+        #     continue
 
         command = (
             f"{executable_path} -S "

src/backend/riscv64/peephole.rs

@@ -1035,18 +1035,22 @@ pub fn remove_redundant_labels(mctx: &mut MContext<RvInst>) -> bool {
     changed
 }
 
-pub fn run_peephole(mctx: &mut MContext<RvInst>, config: &LowerConfig) -> bool {
+pub fn run_peephole(mctx: &mut MContext<RvInst>, config: &LowerConfig, aggressive: bool) -> bool {
     let mut runner1 = PeepholeRunner::new();
     let mut runner2 = PeepholeRunner::new();
     let mut runner3 = PeepholeRunner::new();
 
     runner1.add_rule(li_dce());
     runner1.add_rule(remove_identity_move());
     runner1.add_rule(remove_redundant_move());
-    runner1.add_rule(remove_redundant_move_word()); // aggressive
+    if aggressive {
+        runner1.add_rule(remove_redundant_move_word());
+    }
 
     runner2.add_rule(fuse_cmp_br());
-    runner2.add_rule(fuse_fmul_faddfsub()); // aggressive
+    if aggressive {
+        runner2.add_rule(fuse_fmul_faddfsub());
+    }
     runner2.add_rule(fuse_sub_br());
 
     runner3.add_rule(fuse_xori_cmp_br());

src/backend/riscv64/schedule.rs

@@ -6,13 +6,7 @@ use std::{
 
 use super::inst::{AluOpRRR, FpuOpRR, FpuOpRRR, FpuOpRRRR, RvInst, RvInstKind};
 use crate::{
-    backend::{
-        inst::{DisplayMInst, MInst},
-        LowerConfig,
-        MBlock,
-        MContext,
-        MFunc,
-    },
+    backend::{inst::MInst, LowerConfig, MBlock, MContext, MFunc},
     collections::linked_list::{LinkedListContainerPtr, LinkedListNodePtr},
 };
 
@@ -882,7 +876,7 @@ fn shedule_chunk(mctx: &mut MContext<RvInst>, start: RvInst, end: RvInst, config
             s += 1;
         }
 
-        println!("scheduling: {}, s: {}", n.display(mctx), s);
+        // println!("scheduling: {}, s: {}", n.display(mctx), s);
 
         scheduled.insert(n, s);