Skip to content

Commit

Permalink
Merge pull request #50 from ut-issl/feature/cpp17
Browse files Browse the repository at this point in the history
Move to C++17!!!
  • Loading branch information
sksat authored Feb 8, 2022
2 parents 160d837 + 1608c38 commit 9b86700
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 2 deletions.
2 changes: 2 additions & 0 deletions .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,8 @@ on:
- '.github/workflows/build.yml'
- 'ExtLibraries/**'
- 'CMakeLists.txt'
- 'common.cmake'
- 'CMakeSettings.json'
- 'src/**'

jobs:
Expand Down
1 change: 0 additions & 1 deletion CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,6 @@ option(USE_C2A "Use C2A" OFF)
# preprocessor
if(WIN32)
add_definitions(-DWIN32)
#add_definitions(-D_HAS_STD_BYTE=0) #when you chose C++ 17
endif()

## set directory path
Expand Down
3 changes: 2 additions & 1 deletion common.cmake
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
# C++
set_target_properties(${PROJECT_NAME} PROPERTIES LANGUAGE CXX)
set_target_properties(${PROJECT_NAME} PROPERTIES CXX_STANDARD 14)
set_target_properties(${PROJECT_NAME} PROPERTIES CXX_STANDARD 17)
set_target_properties(${PROJECT_NAME} PROPERTIES CXX_EXTENSIONS FALSE)

# common include
target_include_directories(${PROJECT_NAME} PUBLIC ${S2E_DIR}/src)
Expand Down

0 comments on commit 9b86700

Please sign in to comment.