Skip to content

Commit

Permalink
Add full tests to makefile (#18)
Browse files Browse the repository at this point in the history
* Add ability to run check-radsan from makefile

* update readme and makefile

* Go back to just `test` target, bump llvm version

* Fix README
  • Loading branch information
cjappl authored Dec 19, 2023
1 parent b46e3e0 commit 7a173bc
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 5 deletions.
7 changes: 3 additions & 4 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -52,8 +52,7 @@ test-targets: generate
cmake --build $(BUILD_DIR) --target TRadsan-$(ARCH)-Test TRadsan-$(ARCH)-NoInstTest -j$(NPROCS)

test: test-targets
$(BUILD_DIR)/projects/compiler-rt/lib/radsan/tests/Radsan-$(ARCH)-NoInstTest
$(BUILD_DIR)/projects/compiler-rt/lib/radsan/tests/Radsan-$(ARCH)-Test
cmake --build $(BUILD_DIR) --target check-radsan -j$(NPROCS)

check-compiler-rt: generate
cmake --build $(BUILD_DIR) --target check-compiler-rt -j$(NPROCS)
Expand All @@ -73,9 +72,9 @@ help:
@echo " help Show this help message"
@echo " generate Generate build files"
@echo " clang Build clang and compiler-rt"
@echo " check-compiler-rt Run all compiler-rt tests (extremely slow)"
@echo " check-compiler-rt Run all compiler-rt tests (extremely slow, may have failures)"
@echo " test-targets Build radsan tests"
@echo " test Build and run radsan tests"
@echo " test Run radsan tests"
@echo " docker Build docker image"
@echo " clean Clean build directory"
@echo ""
Expand Down
1 change: 1 addition & 0 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -298,6 +298,7 @@ targets to the compiler-rt project for each architecture `arch`:

1. `TRadsan-${arch}-Test` (which is instrumented by RADSan), and
2. `TRadsan-${arch}-NoInstTest` (which are unit tests that do not need the RADSan instrumentation)
3. The "lit tests" which compile and run tests programs from scratch (see llvm-project/compiler-rt/test/radsan/TestCases/)

To run the tests:

Expand Down

0 comments on commit 7a173bc

Please sign in to comment.