From 1ba664499a67d3071830d2dc3a3433b9ca0d28a1 Mon Sep 17 00:00:00 2001 From: Michal Czyz Date: Fri, 29 Mar 2024 13:07:46 +0100 Subject: [PATCH] Test run without implementation Signed-off-by: Michal Czyz --- .github/workflows/xls-modules.yml | 74 +++++++++++++++++++------------ 1 file changed, 45 insertions(+), 29 deletions(-) diff --git a/.github/workflows/xls-modules.yml b/.github/workflows/xls-modules.yml index 5255da08bd..b4bc15f8c6 100644 --- a/.github/workflows/xls-modules.yml +++ b/.github/workflows/xls-modules.yml @@ -40,8 +40,8 @@ jobs: uses: actions/cache@v4 with: path: "~/.cache/bazel" - key: ${{env.CACHE_KEY}} - restore-keys: ${{env.CACHE_RESTORE_KEY}} + key: ${{ env.CACHE_KEY }} + restore-keys: ${{ env.CACHE_RESTORE_KEY }} - name: Increase build space run: | @@ -73,9 +73,8 @@ jobs: //xls/dslx:interpreter_main \ //xls/dslx/ir_convert:ir_converter_main \ //xls/tools:opt_main \ - //xls/tools:codegen_main - - + //xls/tools:codegen_main \ + //xls/dslx:dslx_fmt config-matrix: needs: build @@ -109,19 +108,35 @@ jobs: steps: - uses: actions/checkout@v4 - - name: Free disk - id: free-disk - uses: ./.github/actions/free-disk + - name: Increase build space + run: | + echo "Before cleanup" + df -H + sudo rm -rf /usr/share/dotnet/* + sudo rm -rf /usr/local/lib/android/* + sudo rm -rf /usr/share/dotnet + sudo rm -rf /opt/ghc + sudo rm -rf "/usr/local/share/boost" + sudo rm -rf "$AGENT_TOOLSDIRECTORY" + echo "After cleanup" + df -H - - name: Implement CSR - id: implementation - uses: ./.github/actions/implementation + - name: Bazel Cache + uses: actions/cache@v4 with: - xls_module: ${{ matrix.module.xls_module }} - rule_ir: ${{ matrix.module.rule_ir }} - rule_verilog: ${{ matrix.module.rule_verilog }} - rule_synthesis: ${{ matrix.module.rule_synthesis }} - rule_pnr: ${{ matrix.module.rule_pnr }} + path: "~/.cache/bazel" + key: ${{env.CACHE_KEY}} + restore-keys: ${{env.CACHE_RESTORE_KEY}} + + # - name: Implement CSR + # id: implementation + # uses: ./.github/actions/implementation + # with: + # xls_module: ${{ matrix.module.xls_module }} + # rule_ir: ${{ matrix.module.rule_ir }} + # rule_verilog: ${{ matrix.module.rule_verilog }} + # rule_synthesis: ${{ matrix.module.rule_synthesis }} + # rule_pnr: ${{ matrix.module.rule_pnr }} test: needs: build @@ -142,18 +157,16 @@ jobs: steps: - uses: actions/checkout@v4 - - name: Free disk - id: free-disk - uses: ./.github/actions/free-disk - - - name: Test ${{ matrix.dslx_test }} - id: test - uses: ./.github/actions/test + - name: Bazel Cache + uses: actions/cache@v4 with: - xls_module: "xls/modules/dma" - rule_test: ${{ matrix.dslx_test }} - + path: "~/.cache/bazel" + key: ${{env.CACHE_KEY}} + restore-keys: ${{env.CACHE_RESTORE_KEY}} + - name: Test + run: | + bazel run -c opt --test_output=errors -- ${{env.XLS_MODULE}}:${{ matrix.dslx_test }} format: name: Format @@ -162,9 +175,12 @@ jobs: steps: - uses: actions/checkout@v4 - - name: Build formatter - run: | - bazel build -c opt --test_output=errors -- //xls/dslx:dslx_fmt + - name: Bazel Cache + uses: actions/cache@v4 + with: + path: "~/.cache/bazel" + key: ${{env.CACHE_KEY}} + restore-keys: ${{env.CACHE_RESTORE_KEY}} # Once https://github.com/google/xls/issues/1285 is implemented, # we could replace these with a single rule