Skip to content

Commit

Permalink
Merge pull request #1257 from DARMA-tasking/1256-move-vt-trace-option…
Browse files Browse the repository at this point in the history
…-before-target-creation

1256:  Create vt_trace_only option before creating trace-only CMake target
  • Loading branch information
JacobDomagala authored Feb 16, 2021
2 parents 007e872 + 37d5db2 commit 8902f8e
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 7 deletions.
5 changes: 5 additions & 0 deletions CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -110,6 +110,11 @@ include(cmake/load_bundled_libraries.cmake)

include(cmake/nvcc_no_deprecated_gpu_targets.cmake)

option(vt_trace_only "Build VT with trace-only mode enabled" OFF)
if (vt_trace_only)
message(STATUS "Building additional target for VT in trace-only mode")
endif()

# Primary VT build
add_subdirectory(src)

Expand Down
5 changes: 0 additions & 5 deletions cmake/define_build_types.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -104,11 +104,6 @@ else()
set(vt_feature_cmake_trace_enabled "0")
endif()

option(vt_trace_only "Build VT with trace-only mode enabled" ON)
if (vt_trace_only)
message(STATUS "Building additional target for VT in trace-only mode")
endif()

# This will be set to 1 during generation of cmake config for vt-trace target
set(vt_feature_cmake_trace_only "0")

Expand Down
3 changes: 1 addition & 2 deletions src/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -252,13 +252,12 @@ link_target_with_vt(
DEFAULT_LINK_SET
)

# Define and set up the different build types of VT

include(../cmake/trace_only_functions.cmake)
if (vt_trace_only)
create_trace_only_target()
endif()

# Define and set up the different build types of VT
include(../cmake/define_build_types.cmake)

target_include_directories(
Expand Down

0 comments on commit 8902f8e

Please sign in to comment.