From bdc33e490c21106e1a50faae4cf6c9dfd92ae9e3 Mon Sep 17 00:00:00 2001
From: Jiuyang Liu <liu@jiuyang.me>
Date: Thu, 1 Aug 2024 03:52:37 +0800
Subject: [PATCH] [ci] dont run verilator on PR CI

---
 .github/workflows/trace.yml     | 17 +----------------
 .github/workflows/verilator.yml | 19 +++++++++++++++++--
 2 files changed, 18 insertions(+), 18 deletions(-)

diff --git a/.github/workflows/trace.yml b/.github/workflows/trace.yml
index 07d675386..a7efb8e16 100644
--- a/.github/workflows/trace.yml
+++ b/.github/workflows/trace.yml
@@ -42,19 +42,4 @@ jobs:
           ref: ${{ github.event.pull_request.head.sha }}
       - name: "Build vcs emulator"
         run: |
-          nix build '.#t1.${{ matrix.config }}.ip.vcs-emu-trace' --impure --no-link --cores 64
-
-  # In the future, we may choose Verdi for trace, and left verilator trace only for performance evaluation
-  build-verilator-trace-emulators:
-    name: "Build verilator trace emulators"
-    needs: [gen-test-plan]
-    runs-on: [self-hosted, linux, nixos, BIGRAM]
-    strategy:
-      fail-fast: false
-      matrix: ${{ fromJSON(needs.gen-test-plan.outputs.testplan) }}
-    steps:
-      - uses: actions/checkout@v4
-        with:
-          ref: ${{ github.event.pull_request.head.sha }}
-      - name: "Build verilator emulator with trace"
-        run: nix build '.#t1.${{ matrix.config }}.ip.verilator-emu-trace' -L --no-link --cores 64
\ No newline at end of file
+          nix build '.#t1.${{ matrix.config }}.ip.vcs-emu-trace' --impure --no-link --cores 64
\ No newline at end of file
diff --git a/.github/workflows/verilator.yml b/.github/workflows/verilator.yml
index 339cccc43..7573304ab 100644
--- a/.github/workflows/verilator.yml
+++ b/.github/workflows/verilator.yml
@@ -17,7 +17,7 @@ jobs:
       - id: "get-all-configs"
         run: echo "out=$(nix run .#ci-helper generateTestPlan)" > $GITHUB_OUTPUT
 
-  build-emulators:
+  build-verilator-emulators:
     name: "Build Verilator Emulators"
     needs: [gen-test-plan]
     runs-on: [self-hosted, linux, nixos, BIGRAM]
@@ -36,9 +36,24 @@ jobs:
           # Build testcases with vlen 1024 and vlen 4096
           nix build ".#t1.${{ matrix.config }}.ip.cases.all" --max-jobs auto -L --no-link --cores 64
 
+  # In the future, we may choose Verdi for trace, and left verilator trace only for performance evaluation
+  build-verilator-trace-emulators:
+    name: "Build verilator trace emulators"
+    needs: [gen-test-plan]
+    runs-on: [self-hosted, linux, nixos, BIGRAM]
+    strategy:
+      fail-fast: false
+      matrix: ${{ fromJSON(needs.gen-test-plan.outputs.testplan) }}
+    steps:
+      - uses: actions/checkout@v4
+        with:
+          ref: ${{ github.event.pull_request.head.sha }}
+      - name: "Build verilator emulator with trace"
+        run: nix build '.#t1.${{ matrix.config }}.ip.verilator-emu-trace' -L --no-link --cores 64
+
   gen-matrix:
     name: "Prepare for running testcases"
-    needs: [build-emulators]
+    needs: [build-verilator-emulators]
     runs-on: [self-hosted, linux, nixos, BIGRAM]
     env:
       RUNNERS: 70