Skip to content

Commit

Permalink
Fix assertion error when debug=yes and optimize=yes (#810)
Browse files Browse the repository at this point in the history
  • Loading branch information
yjf2002ghty authored Jul 10, 2024
1 parent ddfcac4 commit 81dd96c
Show file tree
Hide file tree
Showing 2 changed files with 14 additions and 2 deletions.
14 changes: 13 additions & 1 deletion .github/workflows/stockfish.yml
Original file line number Diff line number Diff line change
Expand Up @@ -46,7 +46,7 @@ jobs:
- name: Download required packages
run: |
sudo apt update
sudo apt install expect valgrind g++-multilib
sudo apt install expect valgrind g++-multilib llvm
- name: Download the used network from the fishtest framework
run: |
Expand Down Expand Up @@ -74,6 +74,12 @@ jobs:
make -j2 ARCH=x86-32 optimize=no debug=yes build
../tests/signature.sh $benchref
- name: Test debug x86-32 profile build (Optimization on, All features)
run: |
export CXXFLAGS=""
make clean
make -j2 ARCH=x86-32 optimize=yes all=yes largeboards=yes debug=yes profile-build
- name: Test x86-32 build
run: |
make clean
Expand Down Expand Up @@ -107,6 +113,12 @@ jobs:
make -j2 ARCH=x86-64-modern optimize=no debug=yes build
../tests/signature.sh $benchref
- name: Test debug x86-64-modern profile build (Optimization on, All features)
run: |
export CXXFLAGS=""
make clean
make -j2 ARCH=x86-64-modern optimize=yes all=yes largeboards=yes debug=yes profile-build
- name: Test x86-64-modern build
run: |
make clean
Expand Down
2 changes: 1 addition & 1 deletion src/Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -524,7 +524,7 @@ endif
### 3.3 Optimization
ifeq ($(optimize),yes)

CXXFLAGS += -O3
CXXFLAGS += -O3 -fno-strict-aliasing

ifeq ($(comp),gcc)
ifeq ($(OS), Android)
Expand Down

0 comments on commit 81dd96c

Please sign in to comment.