Skip to content

Commit

Permalink
Build ubench in CI (#305)
Browse files Browse the repository at this point in the history
### Issue
N/A

### Description
tests/microbenchmark wasn't getting built by CI

### List of the changes
modify CI yaml to build it with the unit tests

### Testing
Builds locally, CI test pending

### API Changes
There are no API changes in this PR.
  • Loading branch information
joelsmithTT authored Nov 23, 2024
1 parent 0a41bf5 commit 59d5b51
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 6 deletions.
6 changes: 3 additions & 3 deletions .github/workflows/build-tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ on:


env:
BUILD_TARGET: umd_tests
BUILD_TARGETS: "umd_tests ubench"
BUILD_OUTPUT_DIR: ./build
LIB_OUTPUT_DIR: ./build/lib
DEPS_OUTPUT_DIR: ./build/_deps
Expand Down Expand Up @@ -62,11 +62,11 @@ jobs:
submodules: recursive
lfs: 'true'

- name: Build ${{ env.BUILD_TARGET }}
- name: Build ${{ env.BUILD_TARGETS }}
run: |
echo "Compiling the code..."
cmake -B ${{ env.BUILD_OUTPUT_DIR }} -G Ninja -DTT_UMD_BUILD_TESTS=ON
cmake --build ${{ env.BUILD_OUTPUT_DIR }} --target ${{ env.BUILD_TARGET }}
cmake --build ${{ env.BUILD_OUTPUT_DIR }} --target ${{ env.BUILD_TARGETS }}
echo "Compile complete."
# This is needed to preserve file permissions
Expand Down
4 changes: 1 addition & 3 deletions tests/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -58,9 +58,7 @@ else()
include_directories("${CMAKE_CURRENT_BINARY_DIR}/$ENV{ARCH_NAME}")
endif()

if(MASTER_PROJECT)
add_subdirectory(${CMAKE_CURRENT_SOURCE_DIR}/microbenchmark)
endif()
add_subdirectory(${CMAKE_CURRENT_SOURCE_DIR}/microbenchmark)
add_subdirectory(${CMAKE_CURRENT_SOURCE_DIR}/api)
add_subdirectory(${CMAKE_CURRENT_SOURCE_DIR}/pcie)
add_subdirectory(${CMAKE_CURRENT_SOURCE_DIR}/simulation)
Expand Down

0 comments on commit 59d5b51

Please sign in to comment.