Skip to content

Commit

Permalink
check if windows builds with atomics switch on msvc
Browse files Browse the repository at this point in the history
  • Loading branch information
radkesvat committed Mar 28, 2024
1 parent 79d6365 commit 14853ae
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 47 deletions.
13 changes: 6 additions & 7 deletions .github/workflows/cmake-multi-platform.yml
Original file line number Diff line number Diff line change
Expand Up @@ -30,19 +30,18 @@ jobs:
- os: windows-latest
c_compiler: cl
cpp_compiler: cl
- os: ubuntu-latest
- os: windows-latest
c_compiler: gcc
cpp_compiler: g++
- os: ubuntu-latest
- os: windows-latest
c_compiler: clang
cpp_compiler: clang++
exclude:
- os: windows-latest
- os: ubuntu-latest
c_compiler: gcc
- os: windows-latest
c_compiler: clang
cpp_compiler: g++
- os: ubuntu-latest
c_compiler: cl
c_compiler: clang
cpp_compiler: clang++

steps:
- uses: actions/checkout@v3
Expand Down
39 changes: 0 additions & 39 deletions .github/workflows/cmake-single-platform.yml

This file was deleted.

2 changes: 1 addition & 1 deletion CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -289,7 +289,7 @@ target_compile_definitions(Waterwall PRIVATE DEBUG=1)
endif()

add_compile_options(
$<$<CXX_COMPILER_ID:MSVC>:/W4 /WX>
$<$<CXX_COMPILER_ID:MSVC>:/W4 /WX> /experimental:c11atomics
$<$<NOT:$<CXX_COMPILER_ID:MSVC>>:-Wall -Wextra -Wpedantic -Werror>
)

Expand Down

0 comments on commit 14853ae

Please sign in to comment.