Skip to content

Commit

Permalink
ci: tests should be built without -O0
Browse files Browse the repository at this point in the history
Tests should make sure real use cases work as expected. Some tests might
not working correctly if -O0 is used as optimization level in the
compiler. For example, the ROCm runtime submits a kernel of 4 waves if
the tests are built using -O0, which makes the tests fail.
  • Loading branch information
gcongiu committed Dec 1, 2023
1 parent ef1cc48 commit b1420eb
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/ci.sh
Original file line number Diff line number Diff line change
Expand Up @@ -43,9 +43,9 @@ if [ "$COMPONENT" = "infiniband_umad" ]; then
fi

if [ "$COMPONENT" = "perf_event" ]; then
./configure --with-debug=yes --enable-warnings
./configure --enable-warnings
else
./configure --with-debug=yes --enable-warnings --with-components=$COMPONENT
./configure --enable-warnings --with-components=$COMPONENT
fi

make -j4
Expand Down

0 comments on commit b1420eb

Please sign in to comment.